-
Notifications
You must be signed in to change notification settings - Fork 198
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
fix(picker): update S2 border-color [SWC-641] #3584
Conversation
🦋 Changeset detectedLatest commit: 7f4d9f8 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 |
File metricsSummaryTotal size: 2.25 MB*
picker
* An ASCII character in UTF-8 is 8 bits or 1 byte. |
🚀 Deployed on https://pr-3584--spectrum-css.netlify.app |
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.
Please update the changeset as described and update with the latest to resolve the conflict in the spectrum-two file.
.changeset/fuzzy-houses-clap.md
Outdated
"@spectrum-css/picker": patch | ||
--- | ||
|
||
chore: upgrade picker border-color |
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.
This needs to be more detailed about what properties were specifically updated and what the impact might be for consumers.
dc74716
to
a2d64f8
Compare
@castastrophe I have added a couple of more design review updates for background color and border color for |
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! Thanks 🥳
"@spectrum-css/picker": patch | ||
--- | ||
|
||
Previously, the --spectrum-picker-border-color-default-open was gray-500, but the spec shows it as gray-800. This change aligns the component with the spec by updating the --spectrum-picker-border-color-default-open to gray-800. |
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.
@Rajdeepc Should we add a quick note here about the background color update for the s2 theme?
9707291
to
4f57a79
Compare
4f57a79
to
7f4d9f8
Compare
Description
Updated border-color tokens for picker.
https://jira.corp.adobe.com/browse/SWC-641
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
Include steps for the PR reviewer that explain how they should test your PR. Example test outline:
var(--spectrum-gray-800)
using inspect element by searching for--system-picker-border-color-default-open
var(--spectrum-gray-800)
and applied [@castastrophe]open not hover
variant isvar(--spectrum-gray-900)
using inspect element by searching for--system-picker-border-color-default-open:
@marissahuysentruytopen not hover
variant isvar(--spectrum-gray-200)
using inspect element by searching for--system-picker-background-color-default-open
@marissahuysentruytRegression testing
Validate:
Screenshots
The one on the top is with


--system-picker-border-color-default-open: var(--spectrum-gray-800)
(Desired)The one on the bottom is with
--system-picker-border-color-default-open: var(--spectrum-gray-500)
(Current)To-do list