Skip to content

Improve accessibility: focus rings, contrast, tablist/form semantics - #25

Merged
rastermanden merged 2 commits into
mainfrom
claude/website-review-V4jvm
Apr 20, 2026
Merged

Improve accessibility: focus rings, contrast, tablist/form semantics#25
rastermanden merged 2 commits into
mainfrom
claude/website-review-V4jvm

Conversation

@rastermanden

Copy link
Copy Markdown
Owner
  • Add global :focus-visible outline, honor prefers-reduced-motion, and a
    disabled state for .btn in style.css
  • Raise secondary-text opacity from 0.5–0.6 to 0.78–0.9 on paper-background
    sections to clear WCAG AA contrast
  • Promote the member tab strip to role=tablist with aria-selected,
    arrow/Home/End keyboard navigation, and labelled tabpanel
  • Mark Exchange mode toggles with aria-pressed, label the amount input,
    and announce the converted result via aria-live
  • Associate Newsletter form labels via for/id, add autocomplete=email,
    and bump fine-print contrast; announce submission success politely

claude added 2 commits April 20, 2026 15:07
- Add global :focus-visible outline, honor prefers-reduced-motion, and a
  disabled state for .btn in style.css
- Raise secondary-text opacity from 0.5–0.6 to 0.78–0.9 on paper-background
  sections to clear WCAG AA contrast
- Promote the member tab strip to role=tablist with aria-selected,
  arrow/Home/End keyboard navigation, and labelled tabpanel
- Mark Exchange mode toggles with aria-pressed, label the amount input,
  and announce the converted result via aria-live
- Associate Newsletter form labels via for/id, add autocomplete=email,
  and bump fine-print contrast; announce submission success politely
Runs only when CLAUDE_CODE_REMOTE=true. Installs npm deps if the
lockfile is newer than node_modules, then provisions headless
Chromium via playwright install --with-deps, falling back to a
plain browser install if the apt step fails.

Unignore .claude/settings.json and .claude/hooks/ so the config
is shared across sessions; keep settings.local.json ignored.
@rastermanden
rastermanden merged commit 673a8e0 into main Apr 20, 2026
1 check passed
@rastermanden
rastermanden deleted the claude/website-review-V4jvm branch April 20, 2026 15:30
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