Skip to content

fix(a11y): add missing ARIA props across codebase#8

Merged
crippledgeek merged 5 commits into
masterfrom
chore/aria-accessibility-fixes
Mar 4, 2026
Merged

fix(a11y): add missing ARIA props across codebase#8
crippledgeek merged 5 commits into
masterfrom
chore/aria-accessibility-fixes

Conversation

@crippledgeek
Copy link
Copy Markdown
Owner

Summary

  • Pipe TanStack Form validation errors into React Aria's errorMessage prop on SelectField and TextAreaField, enabling automatic aria-describedby wiring via React Aria's built-in FieldError
  • Remove unused FieldErrorMessage component (replaced by React Aria's native error handling)
  • Add aria-live="polite" region around converter output so screen readers announce results
  • Add aria-label to search input, aria-pressed to category filter buttons, aria-hidden to decorative icons and emojis on Home page
  • Add role="status" and aria-label to all Loader variants, aria-hidden to spinner icons

Test plan

  • Build passes (npm run build)
  • Lint passes (npm run lint)
  • Verify screen reader announces converter output after conversion
  • Verify form validation errors are associated with inputs (check aria-describedby in DevTools)
  • Verify category filter buttons report pressed state to assistive technology
  • Verify loader components announce "Loading" status to screen readers

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 4, 2026

Deploy Preview for devkit-app ready!

Name Link
🔨 Latest commit f967ec1
🔍 Latest deploy log https://app.netlify.com/projects/devkit-app/deploys/69a881e3c5c5ba0008b69abc
😎 Deploy Preview https://deploy-preview-8--devkit-app.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@crippledgeek crippledgeek merged commit 910fc6d into master Mar 4, 2026
6 checks passed
@crippledgeek crippledgeek deleted the chore/aria-accessibility-fixes branch March 4, 2026 20:45
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.

1 participant