Skip to content

fix(tabs): remove close hover fill - #22

Merged
helbertm merged 1 commit into
masterfrom
codex/tab-action-hover-inset
Jul 30, 2026
Merged

fix(tabs): remove close hover fill#22
helbertm merged 1 commit into
masterfrom
codex/tab-action-hover-inset

Conversation

@helbertm

@helbertm helbertm commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • preserve the 26px close/rename hit targets
  • remove hover and focus background fills from inline tab actions
  • color only the close X with the danger color on hover
  • keep the external keyboard focus ring

Validation

  • npm run validate:full:ci
  • npm run validate:browser:quality
  • npm run validate:native:chromium
  • Da Vinci release checkpoint: approved with no blocking findings

@helbertm
helbertm force-pushed the codex/tab-action-hover-inset branch from 427f6c4 to 55e9dc3 Compare July 30, 2026 17:48
@helbertm helbertm changed the title fix(tabs): inset inline action feedback fix(tabs): remove close hover fill Jul 30, 2026
@helbertm
helbertm merged commit d44f759 into master Jul 30, 2026
7 checks passed
@helbertm
helbertm deleted the codex/tab-action-hover-inset branch July 30, 2026 17:49
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