Skip to content

Conversation

@JustasMonkev
Copy link

When aria-errormessage contains multiple space-separated IDs, the check now correctly verifies that all IDs are present in aria-describedby instead of failing to match the entire string against tokenized values.

Closes: #4957

@JustasMonkev JustasMonkev requested a review from a team as a code owner December 27, 2025 06:04
@CLAassistant
Copy link

CLAassistant commented Dec 27, 2025

CLA assistant check
All committers have signed the CLA.

@JustasMonkev JustasMonkev changed the title fix(checks): handle multiple ID tokens in aria-errormessage with aria… fix(checks): handle multiple ID tokens in aria-errormessage Dec 27, 2025
@JustasMonkev JustasMonkev changed the title fix(checks): handle multiple ID tokens in aria-errormessage fix(checks): handle multiple aria-errormessage IDs Dec 27, 2025
@straker
Copy link
Contributor

straker commented Jan 5, 2026

Thanks for the pr. Unfortunately it isn't an accessibility violation if all the aria-describedby ids are not present. As long as one exists then we want to pass this check.

@JustasMonkev
Copy link
Author

Thanks for the pr. Unfortunately it isn't an accessibility violation if all the aria-describedby ids are not present. As long as one exists then we want to pass this check.

Thanks for the comment, updated it

@JustasMonkev
Copy link
Author

@straker could you please check again?

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.

aria-valid-attr-value does not correctly handle multiple id tokens for aria-errormessage with aria-describedby

3 participants