Skip to content

fix(a11y): add aria-describeby and aria-invalid on error messaages - #1263

Open
doppiam wants to merge 1 commit into
italia:mainfrom
doppiam:main
Open

fix(a11y): add aria-describeby and aria-invalid on error messaages#1263
doppiam wants to merge 1 commit into
italia:mainfrom
doppiam:main

Conversation

@doppiam

@doppiam doppiam commented Sep 10, 2026

Copy link
Copy Markdown

PR Checklist

  • My branch is up-to-date with the Upstream main branch.
  • The unit tests pass locally with my changes (if applicable).
  • I have added tests that prove my fix is effective or that my feature works (if applicable).
  • I have added necessary documentation (if appropriate).

Short description of what this resolves:

Improves form error accessibility by exposing invalid states and associating rendered error feedback with the relevant interactive control.

Changes proposed in this Pull Request:

  • fix: add aria-invalid="true" to invalid Input, TextArea and Autocomplete controls
  • fix: assign stable IDs to existing validation messages and reference them via aria-describedby
  • fix: preserve existing help-text and consumer-provided aria-describedby references
  • test: add focused accessibility coverage and update affected Storybook snapshots

@vercel

vercel Bot commented Sep 10, 2026

Copy link
Copy Markdown

@doppiam is attempting to deploy a commit to the dip-trasformazione-digitale Team on Vercel.

A member of the Team first needs to authorize it.

@doppiam doppiam changed the title fix(Input): add aria-describeby and aria-invalid on error messaages fix(a11y): add aria-describeby and aria-invalid on error messaages Sep 10, 2026
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