-
Notifications
You must be signed in to change notification settings - Fork 89
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
[WIP] json for theme 2.0 #3143
base: main
Are you sure you want to change the base?
[WIP] json for theme 2.0 #3143
Conversation
🦋 Changeset detectedLatest commit: 2bb83c3 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
87fbbe9
to
dd534f3
Compare
packages/theme/json/theme.json
Outdated
"opacity": "{foundations.opacity.opacity-8}" | ||
}, | ||
"color-white-8a": { | ||
"$value": "#ffffff00", |
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.
unsure where opacity stands in theme 2.0, if we would link to an opacity foundation here, and have color and opacity as separate values like the JSON structure for theme 1 fade tokens @origami-z
} | ||
} | ||
|
||
function validate() { |
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.
@origami-z this should check all paths are correct refs
cae97df
to
6952b0f
Compare
/release-pr |
🫰✨ Thanks @libertymayc! Your snapshots have been published to npm. Test the snapshots by updating your yarn add @salt-ds/core@0.0.0-snapshot-20240320173647 yarn add @salt-ds/lab@0.0.0-snapshot-20240320173647 yarn add @salt-ds/theme@0.0.0-snapshot-20240320173647 |
Storybook Preview Link https://c21ee998.saltdesignsystem-storybook.pages.dev |
/release-pr |
🫰✨ Thanks @libertymayc! Your snapshots have been published to npm. Test the snapshots by updating your yarn add @salt-ds/core@0.0.0-snapshot-20240320194833 yarn add @salt-ds/lab@0.0.0-snapshot-20240320194833 yarn add @salt-ds/theme@0.0.0-snapshot-20240320194833 |
/release-pr |
🫰✨ Thanks @libertymayc! Your snapshots have been published to npm. Test the snapshots by updating your yarn add @salt-ds/core@0.0.0-snapshot-20240320195607 yarn add @salt-ds/lab@0.0.0-snapshot-20240320195607 yarn add @salt-ds/theme@0.0.0-snapshot-20240320195607 |
229eddc
to
8655e8c
Compare
/release-pr |
🫰✨ Thanks @libertymayc! Your snapshots have been published to npm. Test the snapshots by updating your yarn add @salt-ds/core@0.0.0-snapshot-20240422144812 yarn add @salt-ds/theme@0.0.0-snapshot-20240422144812 |
200c7fd
to
8fd8f1c
Compare
🫰✨ Thanks @libertymayc! Your snapshots have been published to npm. Test the snapshots by updating your yarn add @salt-ds/core@0.0.0-snapshot-20240517153906 yarn add @salt-ds/lab@0.0.0-snapshot-20240517153906 yarn add @salt-ds/theme@0.0.0-snapshot-20240517153906 |
…irs. foundations only override if -next file is provided
3f902cd
to
2bb83c3
Compare
/release-pr |
🫰✨ Thanks @libertymayc! Your snapshots have been published to npm. Test the snapshots by updating your yarn add @salt-ds/core@0.0.0-snapshot-20240517155318 yarn add @salt-ds/lab@0.0.0-snapshot-20240517155318 yarn add @salt-ds/theme@0.0.0-snapshot-20240517155318 |
f260109
to
09d4ce5
Compare
Some quick thoughts for future myself while starting #4041, trying to think how we can best leverage tools out there, e.g. v4 of style-dictionary supports DTCG now.
|
Builds from color-palette-next branch to use salt theme 2.0