Skip to content

style: use semantic focus-ring token across components#2167

Closed
pixelflips wants to merge 4 commits intodevelopfrom
style/focus-ring-tokens
Closed

style: use semantic focus-ring token across components#2167
pixelflips wants to merge 4 commits intodevelopfrom
style/focus-ring-tokens

Conversation

@pixelflips
Copy link
Member

Summary

  • Replace hardcoded var(--pine-color-purple-600) with var(--pine-color-focus-ring) in 10 component stylesheets
  • Add missing focus ring to breadcrumbs progressbar variant (was showing default browser outline)

Closes DSS-91

Components Updated

button, choice, link, dropdown, pagination, breadcrumbs, nav, alert, status_icon, media_tile, hero

Test plan

  • Tab through each component to verify :focus-visible ring renders correctly
  • Confirm breadcrumbs progressbar variant shows focus ring without breaking underline
  • Confirm danger focus rings are unchanged
  • Confirm no remaining purple-600 in focus-ring contexts

prsimp and others added 4 commits February 11, 2026 11:47
Remove GetDX service identification and deployment reporting
integrations as part of the Shutter DX project (PE-2690).

Co-authored-by: Cursor <cursoragent@cursor.com>
chore: remove GetDX deployment hook workflows
@pixelflips pixelflips closed this Feb 20, 2026
@pixelflips pixelflips deleted the style/focus-ring-tokens branch February 20, 2026 17:26
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.

2 participants