-
Notifications
You must be signed in to change notification settings - Fork 194
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(card,contextualhelp,swatch,tokens): add theme tokens for components to global scope #3253
Conversation
🦋 Changeset detectedLatest commit: c3246ca The changes in this PR will be included in the next version bump. This PR includes changesets to release 5 packages
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 |
🚀 Deployed on https://pr-3253--spectrum-css.netlify.app |
File metricsSummaryTotal size: 4.30 MB* Table reports on changes to a package's main file. Other changes can be found in the collapsed Details section below.
Detailscard
contextualhelp
swatch
tokens
* Results are not gzipped or minified. * An ASCII character in UTF-8 is 8 bits or 1 byte. |
2180117
to
ffaafd4
Compare
VRTs were run and showed no changes |
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.
[nit] some of the updates have a larger indentation, but that's non-blocking :)
ffaafd4
to
1ac02fa
Compare
@5t3ph good point! I updated the commit with spacing fixes. |
1ac02fa
to
0ed6328
Compare
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.
Looks good to me — don't see anything out of place or needing to be changed.
0ed6328
to
f0950da
Compare
f0950da
to
384f4e3
Compare
…nts to global scope
384f4e3
to
c9c5f6f
Compare
Description
@spectrum-css/tokens
Add component-level token overrides from card, contextualhelp, swatch, and typography to ensure valid theme toggling.
Light/dark/darkest custom overrides added:
--spectrum-card-selected-background-color-rgb
--spectrum-swatch-border-color
--spectrum-swatch-border-color-light
Medium/large custom overrides added:
--spectrum-contextual-help-content-spacing
Global custom overrides added:
--spectrum-font-family-ar
--spectrum-font-family-he
--spectrum-font-family
--spectrum-font-style
--spectrum-font-size
Component changes
This update removes tokens defined locally that belonged in the global scope. To ensure no regressions, please upgrade your @spectrum-css/tokens package at the same time so you will pick up the component-level definitions now in the global tokens scope. References to
.spectrum--(light|dark|darkest|medium|large)
have been removed.How and where has this been tested?
Please tag yourself on the tests you've marked complete to confirm the tests have been run by someone other than the author.
Validation steps
Regression testing
Validate:
To-do list