Skip to content

fix: no stray focus ring on launch / search click - #262

Merged
omercelikdev merged 1 commit into
mainfrom
fix/stray-focus-ring-on-launch
Jul 13, 2026
Merged

fix: no stray focus ring on launch / search click#262
omercelikdev merged 1 commit into
mainfrom
fix/stray-focus-ring-on-launch

Conversation

@omercelikdev

Copy link
Copy Markdown
Owner

The earlier focus-visible change (#239) didn't cover the real cause.

Root cause

On every window re-open, the OS restores focus to the last-clicked sidebar button, which the browser treats as :focus-visible — so a full orange ring painted around the History icon on launch. The search field also glowed on click via a focus-within ring.

Fix

  • Sidebar buttons — drop the focus-visible ring. The active tab's accent fill already shows the current view, and restored focus can no longer leave a ring on open.
  • Search field — remove the focus-within ring for a static, Spotlight-clean box (the caret is the focus cue).
  • Focus the search input on every window open — so typing filters immediately and focus never sits on a sidebar button to begin with.

tsc clean · 608 tests green · verified live via HMR.

🤖 Generated with Claude Code

The earlier focus-visible change didn't cover the real cause. On every
window re-open the OS restores focus to the last-clicked sidebar button,
which the browser treats as focus-visible — painting a full orange ring
around the History icon on launch. The search field also glowed on click
via a focus-within ring.

- Sidebar buttons: drop the focus-visible ring. The active tab's accent
  fill already signals the current view, and restored focus can no longer
  leave a ring on open.
- Search field: remove the focus-within ring for a static, Spotlight-clean
  box (the caret is the focus cue).
- Focus the search input on every window open, so typing filters
  immediately and focus never sits on a sidebar button to begin with.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@omercelikdev
omercelikdev merged commit 0517c8d into main Jul 13, 2026
@omercelikdev
omercelikdev deleted the fix/stray-focus-ring-on-launch branch July 13, 2026 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant