Skip to content

Fix WCAG AA contrast on active filter chips (History + Report)#261

Merged
ChristopherRotnes merged 2 commits into
masterfrom
claude/history-filter-chip-colors-83H59
May 19, 2026
Merged

Fix WCAG AA contrast on active filter chips (History + Report)#261
ChristopherRotnes merged 2 commits into
masterfrom
claude/history-filter-chip-colors-83H59

Conversation

@ChristopherRotnes
Copy link
Copy Markdown
Owner

Summary

  • Active filter chips in History and Report used --accent (#ee2c80) as fill with white text, giving a contrast ratio of 3.95:1 — failing WCAG AA for normal text (requires 4.5:1 at 11–12px)
  • Added --accent-active: #b5116a token to carbon-tokens.css (~6:1 with white, passes both AA and AAA for normal text)
  • Updated History.jsx active chip background/borderColor to use --accent-active
  • Updated FilterChip in Report.jsx (covers all four filter rows: period, weekdays, session types, instructors) to use --accent-active

Test plan

  • Open History, click a muscle filter chip — verify it renders as darker magenta (not the original hot pink) when active
  • Open Report, activate chips in all four filter rows — same check
  • Verify inactive chips are unchanged (transparent bg, muted border)
  • Check both g100 (dark) and g10 (light) themes — chip active state should look consistent in both

https://claude.ai/code/session_019YEgWSqgreiE9kv145JXkr


Generated by Claude Code

claude added 2 commits May 19, 2026 15:29
Active chips used --accent (#ee2c80) fill with white text, giving 3.95:1
contrast — failing WCAG AA for normal text (requires 4.5:1). Added
--accent-active (#b5116a, ~6:1 with white) token and applied it to
active chip background/border in History.jsx and Report.jsx FilterChip.

https://claude.ai/code/session_019YEgWSqgreiE9kv145JXkr
@github-actions
Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-island-090dfd003-261.westeurope.7.azurestaticapps.net

@ChristopherRotnes ChristopherRotnes marked this pull request as ready for review May 19, 2026 15:53
@ChristopherRotnes ChristopherRotnes merged commit dde4615 into master May 19, 2026
2 checks passed
@ChristopherRotnes ChristopherRotnes deleted the claude/history-filter-chip-colors-83H59 branch May 19, 2026 15:53
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