Skip to content

fix(ui): use accessibility-focus-color for focus styles#16616

Merged
JarrodMFlesch merged 1 commit into
mainfrom
fix/accessibility-focus-color
May 14, 2026
Merged

fix(ui): use accessibility-focus-color for focus styles#16616
JarrodMFlesch merged 1 commit into
mainfrom
fix/accessibility-focus-color

Conversation

@JarrodMFlesch
Copy link
Copy Markdown
Contributor

Consolidates focus styling to use --accessibility-focus-color instead of direct color token references.

Changes

  • Replace --color-border-selected-strong with --accessibility-focus-color in focus states
  • Replace --color-border-selected with --accessibility-focus-color in focus states
  • Update ClearIndicator to use 24px CircledXIcon

Files updated

  • packages/ui/src/elements/CodeEditor/index.css
  • packages/ui/src/elements/Collapsible/index.css
  • packages/ui/src/elements/ReactSelect/index.css
  • packages/ui/src/elements/ReactSelect/ClearIndicator/index.tsx
  • packages/ui/src/elements/ThumbnailCard/index.css
  • packages/ui/src/fields/Checkbox/index.css
  • packages/ui/src/fields/Code/index.css
  • packages/ui/src/fields/DateTime/index.css
  • packages/ui/src/fields/JSON/index.css

- Replace --color-border-selected-strong with --accessibility-focus-color in focus states
- Replace --color-border-selected with --accessibility-focus-color in focus states
- Update ClearIndicator to use 24px CircledXIcon
JessRynkar
JessRynkar previously approved these changes May 14, 2026
@JarrodMFlesch JarrodMFlesch enabled auto-merge (squash) May 14, 2026 14:00
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 14, 2026

📦 esbuild Bundle Analysis for payload

This analysis was generated by esbuild-bundle-analyzer. 🤖
This PR introduced no changes to the esbuild bundle! 🙌

@JarrodMFlesch JarrodMFlesch merged commit 14dabd5 into main May 14, 2026
646 of 657 checks passed
@JarrodMFlesch JarrodMFlesch deleted the fix/accessibility-focus-color branch May 14, 2026 20:39
@PatrikKozak PatrikKozak self-assigned this May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants