-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add support for design tokens #34
base: master
Are you sure you want to change the base?
Conversation
d5462f3
to
a8ef5ae
Compare
2fbec38
to
ef5091d
Compare
@xitij2000 We should add some steps describing the process of updating the tokens and building the theme. |
This change adds support for design tokens to simple theme.
ef5091d
to
03268f5
Compare
89f828a
to
9d3e0b2
Compare
|
||
This is a two-step process: | ||
|
||
1. Building the tokens: The `paragon build-tokens` build tokens command compiles |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1. Building the tokens: The `paragon build-tokens` build tokens command compiles | |
1. Building the tokens: The `npm build-tokens` build tokens command compiles |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The command being the scenes is paragon
. I'm just running it via npm scripts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xitij2000 Yes, I know. People will need to use npm
as they cannot directly run paragon
commands which is why I am suggesting this change.
the JSON style dictionaries into CSS file sin the `paragon/css/` directory. | ||
These are supposed to be committed to the repository. | ||
2. Building the SCSS: The build token CSS files are then compiled into a brand | ||
theme in the dist folder using the `paragon build-scss` command. The contents |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
theme in the dist folder using the `paragon build-scss` command. The contents | |
theme in the dist folder using the `npm build-scss` command. The contents |
Co-authored-by: Navin Karkera <[email protected]>
Co-authored-by: Navin Karkera <[email protected]>
This change adds support for design tokens to simple theme.