Skip to content

Conversation

@Ridsko
Copy link
Collaborator

@Ridsko Ridsko commented Jun 1, 2023

No description provided.

@Ridsko Ridsko force-pushed the overheidnl/theme branch from acbf2c2 to 00a1cfe Compare June 5, 2023 11:32
@Ridsko Ridsko marked this pull request as ready for review June 13, 2023 09:59

/* Theme Design Tokens */
/* TODO: Load themes on-demand */
@import "../../node_modules/@nl-design-system-unstable/overheidnl-design-tokens/dist/index.css";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The development server cannot start with this line active.

@@ -0,0 +1,275 @@
/* Generics global */
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there are still tokens missing or not being set properly. The buttons are missing padding or a white text.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The theme is not loading in because the overheidnl package is not live yet. You need to use npm link on this branch

Base automatically changed from NL-Design-System-Changes to development June 20, 2023 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants