Skip to content

Update: hidden attribute clarifications #507

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 5 commits into from
Dec 13, 2024
Merged

Update: hidden attribute clarifications #507

merged 5 commits into from
Dec 13, 2024
+21 −12

Conversation

scottaohara
Copy link
Member

@scottaohara scottaohara commented Feb 16, 2024 •
edited by pr-preview bot
Loading

Update expectations for use of aria-hidden with the hidden attribute.

Clarify that the use of aria-hidden on an element with the hidden element is generally not recommended - but authors MUST NOT declare these attributes together if the hidden attribute is in the 'until-found' state.

example use case that should result in errors from checkers:

<div hidden aria-hidden=true>
  content that can be found, but will remain inaccessible per aria-hidden attribute
</div>

Preview | Diff

@scottaohara scottaohara added the needs changelog entry once 2 implementations land, we can merge and note it in our changelog. label Feb 16, 2024
scottaohara
scottaohara commented Feb 16, 2024
View reviewed changes
@scottaohara scottaohara marked this pull request as ready for review February 16, 2024 14:48
@scottaohara scottaohara added the needs implementation commitment Cannot merge into spec until implementations in conformance checkers has been confirmed. label Feb 16, 2024
@scottaohara scottaohara added Meets 2 implementations req Indicates that a spec change has met the necessary 2 implementations needed to merge. and removed needs implementation commitment Cannot merge into spec until implementations in conformance checkers has been confirmed. labels Dec 13, 2024
@scottaohara scottaohara removed the needs changelog entry once 2 implementations land, we can merge and note it in our changelog. label Dec 13, 2024
@scottaohara scottaohara merged commit ea7fbdb into gh-pages Dec 13, 2024
2 checks passed
@scottaohara scottaohara deleted the hidden-attribute-clarifications branch December 13, 2024 17:50
@scottaohara scottaohara mentioned this pull request Dec 16, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@Cmac299203 Cmac299203 Cmac299203 approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
Meets 2 implementations req Indicates that a spec change has met the necessary 2 implementations needed to merge.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@scottaohara @Cmac299203