Skip to content

Accessibility: readable count on the selected Projects chip - #249

Merged
darkmastermindz merged 1 commit into
mainfrom
a11y-chip-count
Oct 25, 2026
Merged

darkmastermindz merged 1 commit into
mainfrom
a11y-chip-count

Conversation

@darkmastermindz

Copy link
Copy Markdown
Owner

Summary

CI's a11y spec (from #243) fails on every open PR, and would fail on main too, with one rule on /projects:

color-contrast: .is-on > .chip__count: #aaaaaa on #653496, 3.63:1 (needs 4.5:1)

The count inside the selected filter chip ("All 12") was grey on the purple chip. Its colour is now light purple #e9d5ff, which gives 6.21:1. This sandbox has no network, so /projects has no repos locally, and the problem only showed in CI, which has real data.

Related Issue

Follow-up to #243 (accessibility). It's the failure on #230 and #248: Android touch shards 1/4 and 3/4.

Changes Made

  • pages/projects.vue: .chip.is-on .chip__count { color: #e9d5ff }, with a comment giving the ratio.
  • Unselected chips keep #aaa, which is 6.82:1 on the page background, so they were already fine.
  • The tag chips' selected state (black on green) isn't affected.

How to Test

  • npm test: 253 pass. npm run generate: OK.
  • On the built site, Pixel 7, /projects:
    • the selected chip's count computes to rgb(233, 213, 255) on rgb(101, 52, 150);
    • axe with the WCAG 2.1 A/AA tags finds no violations.
  • tests/e2e/a11y.spec.mjs on Pixel 7 and Galaxy Tab S9 landscape: 28 passed.
  • Ratios were computed with the WCAG formula: #aaa on #653496 is 3.64 (CI measured 3.63), and #e9d5ff on #653496 is 6.21.
  • CI on this PR, with real repos, is the definitive check: Android touch shards 1–4.

Additional Notes


Generated by Claude Code

The count inside the selected filter chip on /projects was grey #aaa on the
purple #653496: 3.6:1, under WCAG's 4.5:1. CI's a11y spec catches it on
every PR now that it has real repos. It's light purple #e9d5ff there now
(6.2:1); unselected chips keep #aaa (6.8:1 on the page).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Nginuw5wmUjUSkmQB1J3pS
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