Skip to content

fix(settings): stabilize navigation hover state - #307

Merged
Tryanks merged 1 commit into
mainfrom
fix/settings-nav-hover
Aug 27, 2026
Merged

fix(settings): stabilize navigation hover state#307
Tryanks merged 1 commit into
mainfrom
fix/settings-nav-hover

Conversation

@Tryanks

@Tryanks Tryanks commented Aug 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep the settings navigation hitbox, stable element ID, hover style, and click handler on the same GPUI element
  • align settings navigation rows with the proven main-sidebar row structure

Root cause

The accessible clickable wrapper and its anonymous hover-painted child created two overlapping interaction regions, allowing hover paint to become stale or skip while crossing adjacent tabs.

Verification

  • cargo check -p tcode-ui
  • cargo test -p tcode-ui (300 passed)
  • cargo fmt --all --check
  • git diff --check

@Tryanks
Tryanks enabled auto-merge (squash) August 27, 2026 20:18
@Tryanks
Tryanks merged commit 08fde7d into main Aug 27, 2026
5 of 6 checks passed
@Tryanks
Tryanks deleted the fix/settings-nav-hover branch August 27, 2026 20:28
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