Fixing Accessibility

fixing-accessibility

Fix accessibility issues.

how to use

• /fixing-accessibility Apply these constraints to any UI work in this conversation.

• /fixing-accessibility <file> Review the file against all rules below and report:

  • violations (quote the exact line or snippet)

  • why it matters (one short sentence)

  • a concrete fix (code-level suggestion)

Do not rewrite large parts of the UI. Prefer minimal, targeted fixes.

when to apply

Reference these guidelines when:

• adding or changing buttons, links, inputs, menus, dialogs, tabs, dropdowns

• building forms, validation, error states, helper text

• implementing keyboard shortcuts or custom interactions

• working on focus states, focus trapping, or modal behavior

• rendering icon-only controls

• adding hover-only interactions or hidden content

rule categories by priority

quick reference

1. accessible names (critical)

• every interactive control must have an accessible name

• icon-only buttons must have aria-label or aria-labelledby

• every input, select, and textarea must be labeled

• links must have meaningful text (no “click here”)

• decorative icons must be aria-hidden

2. keyboard access (critical)

• do not use div or span as buttons without full keyboard support

• all interactive elements must be reachable by Tab

• focus must be visible for keyboard users

• do not use tabindex greater than 0

• Escape must close dialogs or overlays when applicable

3. focus and dialogs (critical)

• modals must trap focus while open

• restore focus to the trigger on close

• set initial focus inside dialogs

• opening a dialog should not scroll the page unexpectedly

4. semantics (high)

• prefer native elements (button, a, input) over role-based hacks

• if a role is used, required aria attributes must be present

• lists must use ul or ol with li

• do not skip heading levels

• tables must use th for headers when applicable

5. forms and errors (high)

• errors must be linked to fields using aria-describedby

• required fields must be announced

• invalid fields must set aria-invalid

• helper text must be associated with inputs

• disabled submit actions must explain why

6. announcements (medium-high)

• critical form errors should use aria-live

• loading states should use aria-busy or status text

• toasts must not be the only way to convey critical information

• expandable controls must use aria-expanded and aria-controls

7. contrast and states (medium)

• ensure sufficient contrast for text and icons

• hover-only interactions must have keyboard equivalents

• disabled states must not rely on color alone

• do not remove focus outlines without a visible replacement

8. media and motion (low-medium)

• images must have correct alt text (meaningful or empty)

• videos with speech should provide captions when relevant

• respect prefers-reduced-motion for non-essential motion

• avoid autoplaying media with sound

9. tool boundaries (critical)

• prefer minimal changes, do not refactor unrelated code

• do not add aria when native semantics already solve the problem

• do not migrate UI libraries unless requested

common fixes

<!-- icon-only button: add aria-label -->
<!-- before --> <button><svg>...</svg></button>
<!-- after -->  <button aria-label="Close"><svg aria-hidden="true">...</svg></button>

<!-- div as button: use native element -->
<!-- before --> <div onclick="save()">Save</div>
<!-- after -->  <button onclick="save()">Save</button>

<!-- form error: link with aria-describedby -->
<!-- before --> <input id="email" /> <span>Invalid email</span>
<!-- after -->  <input id="email" aria-describedby="email-err" aria-invalid="true" /> <span id="email-err">Invalid email</span>

review guidance

• fix critical issues first (names, keyboard, focus, tool boundaries)

• prefer native HTML before adding aria

• quote the exact snippet, state the failure, propose a small fix

• for complex widgets (menu, dialog, combobox), prefer established accessible primitives over custom behavior