Show outline on keyboard nav and style outline to match theme#240
Conversation
Codecov Report
@@ Coverage Diff @@
## master #240 +/- ##
=========================================
Coverage ? 40.07%
=========================================
Files ? 22
Lines ? 272
Branches ? 61
=========================================
Hits ? 109
Misses ? 162
Partials ? 1
Continue to review full report at Codecov.
|
|
@kevjin Feel free to run the following command to build a staging demo whenever there are visual changes to be tested. Just type the following in a comment. It means Google Cloud Build Run.
|
|
/gcbrun |
|
Per
Though, still haven't since the staging URL shown from @ahmadawais's previous |
|
/gcbrun |
|
@ahmadawais weird that |
|
/gcbrun |
1 similar comment
|
/gcbrun |
|
@MylesBorins can you help here? The |
|
github actions doesn't run for PRs from a fork which is why the URL hasn't posted The URLs are deterministic though staging example is https://storage.googleapis.com/staging.nodejs.dev/fa996d0/index.html |
Description
This PR partially reverts the changes made in #224 to comply with a11y on outlines.
Outlines will display during keyboard navigation (ex. tabbing) and will hide on mouse clicks.
Outlines have also been styled using nodejs.dev's brand colors, for better visual appeal.
Default vs custom style comparison
vs.
Resolves #239