Skip to content

fix(button): Restore focus ring on transparent LinkButton#109735

Merged
JonasBa merged 1 commit intomasterfrom
jb/projectpagefilter/items
Mar 3, 2026
Merged

fix(button): Restore focus ring on transparent LinkButton#109735
JonasBa merged 1 commit intomasterfrom
jb/projectpagefilter/items

Conversation

@JonasBa
Copy link
Member

@JonasBa JonasBa commented Mar 2, 2026

Remove box shadow override from LinkButton

LinkButton had an unconditional `box-shadow: none` override on
`:focus-visible` that cancelled the focus ring applied by
`theme.focusRing()` for transparent and link priority buttons.

Non-transparent buttons use a `::after` pseudo-element for their focus
indicator so they are unaffected; transparent buttons rely solely on
the box-shadow from focusRing, which was being zeroed out.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 2, 2026
@JonasBa JonasBa marked this pull request as ready for review March 2, 2026 21:58
@JonasBa JonasBa requested a review from a team as a code owner March 2, 2026 21:59
Copy link
Collaborator

@TkDodo TkDodo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the best PRs 😂

@JonasBa JonasBa merged commit 08312df into master Mar 3, 2026
63 checks passed
@JonasBa JonasBa deleted the jb/projectpagefilter/items branch March 3, 2026 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants