Skip to content

Update: hidden attribute clarifications#507

Merged
scottaohara merged 5 commits intogh-pagesfrom
hidden-attribute-clarifications
Dec 13, 2024
Merged

Update: hidden attribute clarifications#507
scottaohara merged 5 commits intogh-pagesfrom
hidden-attribute-clarifications

Conversation

@scottaohara
Copy link
Member

@scottaohara scottaohara commented Feb 16, 2024 •
edited
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
1 check 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