Site-icon's focus ring is too tall, and is clipped beneath the site icon below it
See original GitHub issueDescription
Focus ring is too tall, and is clipped beneath the site icon below it
Steps to Reproduce
- switch to the
Favorites
view in Top Sites via either the context-menu item or viaCustomize
->Top Sites
->Favorites
- add > 6 favorites (up to 12)
- start pressing
Tab
on the first row, and notice the focus ring
Actual result:
The focus ring extends beneath the next site’s favicon, and looks awkward due to being clipped

Expected result:
No clipping into the next site’s favicon; in Figma, the focus ring looks to merely comprise the favicon/tile, not the accompanying site text

Reproduces how often:
100%
Brave version (brave://version info)
Brave | 1.24.2 Chromium: 89.0.4389.90 (Official Build) nightly (x86_64) |
---|---|
Revision | 62eb262cdaae9ef819aadd778193781455ec7a49-refs/branch-heads/4389@{#1534} |
OS | macOS Version 11.2.3 (Build 20D91) |
Version/Channel Information:
- Can you reproduce this issue with the current release? no
- Can you reproduce this issue with the beta channel? yes
- Can you reproduce this issue with the nightly channel? yes
Issue Analytics
- State:
- Created 3 years ago
- Comments:8 (5 by maintainers)
Top Results From Across the Web
How To Design Useful and Usable Focus Indicators
New from Deque: Learn how to design effective focus indicators that are accessible and comply with WCAG standards for website a11y.
Read more >Buttons - Menus and actions - Human Interface Guidelines
A button always includes a text label or a symbol (or interface icon) — and ... usually within or beneath it — so...
Read more >How-To Create Desktop Shortcut Icons Directly ... - YouTube
Creating desktop shortcuts for a website is a great way to keep your favorite websites organized. In this video we walk you through...
Read more >focus-visible - CSS: Cascading Style Sheets - MDN Web Docs
The :focus-visible pseudo-class applies while an element matches the :focus ... (Many browsers show a "focus ring" by default in this case.)
Read more >Links - Usability & Web Accessibility - Yale University
While screen readers can read a full page to a user, screen reader users ... For such users, it is very important for...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
1.21.x - Not Reproducible 1.22.x - Not Reproducible 1.23.x - Reproducible 1.24.x - Reproducible
Verified
FIXED
usingThe focus ring is now back to spec.