Skip to content
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

Update: hidden attribute clarifications #507

Open
wants to merge 3 commits into
base: gh-pages
Choose a base branch
from
Open

Update: hidden attribute clarifications #507

wants to merge 3 commits into from

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
index.html Outdated Show resolved Hide resolved
@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Cmac299203 Cmac299203 Cmac299203 approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
needs changelog entry once 2 implementations land, we can merge and note it in our changelog. needs implementation commitment Cannot merge into spec until implementations in conformance checkers has been confirmed.
Projects
None yet
Milestone
No milestone
Development

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