-
Notifications
You must be signed in to change notification settings - Fork 201
fix(textfield): update border and background color #3575
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
Conversation
🦋 Changeset detectedLatest commit: 8d61bd6 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 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 |
File metricsSummaryTotal size: 2.25 MB*
textfield
* An ASCII character in UTF-8 is 8 bits or 1 byte. |
🚀 Deployed on https://pr-3575--spectrum-css.netlify.app |
@TarunAdobe Can you please update the branch with the base branch and write out a few validation test cases in the pull request template for us to confirm the updates? |
@TarunAdobe Can you please write out your validation steps for your PRs? See this as an example: #3547 |
172779e
to
746cb62
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.
Confirmed S2 is using gray-800
as per design spec while Legacy is using gray-900
9de29d5
to
1612506
Compare
1612506
to
8d61bd6
Compare
* fix(textfield): update border and background color * fix(swatch): adjust disabled icon color across themes (#3547) * fix(textfield): update border color on keyfocus in s2 --------- Co-authored-by: [ Cassondra ] <castastrophe@users.noreply.github.com> Co-authored-by: rise-erpelding <54716846+rise-erpelding@users.noreply.github.com>
Description
updated border color for key-focus state of textfield in spectrum-two theme.
swc-576
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:
tab
to focus over the textfield and validate the border color isgray-800
as from the design specsRegression testing
Validate:
Screenshots
To-do list