-
Notifications
You must be signed in to change notification settings - Fork 204
fix(infieldbutton): no outline on focus-visible #2288
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
9e6a908
to
59af790
Compare
🚀 Deployed on https://pr-2288--spectrum-css.netlify.app |
File metricsSummaryTotal size: 3.97 MB*
Detailsinfieldbutton
* Results are not gzipped or minified. * An ASCII character in UTF-8 is 8 bits or 1 byte. |
6af1a8a
to
70bbf58
Compare
675de2d
to
8cc732b
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!
@Westbrook mind taking a quick look at this one? Thanks! |
8cc732b
to
117f1ac
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.
LGTM!
117f1ac
to
8562cfc
Compare
8562cfc
to
1dda12e
Compare
1dda12e
to
5a8c3bb
Compare
5a8c3bb
to
d3a131a
Compare
Description
Addresses issue #2232. Per the spec files, the In-field Button component shouldn't have an outline on keyboard focus. Keyboard focus also looks the same as hover focus for this component.
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 @jenndiaz
Regression testing
Validate:
Screenshots
To-do list