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

Consider further rules for aria-hidden #448

Open
scottaohara opened this issue Mar 3, 2023 · 2 comments
Open

Consider further rules for aria-hidden #448

scottaohara opened this issue Mar 3, 2023 · 2 comments
Labels
Allowed aria-* attributes

Comments

@scottaohara
Copy link
Member

As a follow on to #447, this issue is being created in case there are other elements which should disallow the use of aria-hidden on them.

For instance, focusable elements which have not been provided a tabindex=-1 or disabled attribute (noting that for the tabindex=-1 case, that CSS pointer event cancellation would need to also be added, but that may be out of scope for what can reasonably be checked).
@nschonni
Copy link
Contributor

nschonni commented Mar 3, 2023

Would <main> also fall in a similar category, or maybe not anymore, since multiple <main> were made allowed in HTML

@scottaohara
Copy link
Member Author

scottaohara commented Mar 3, 2023 •
edited
Loading

a warning for main, maybe. but there could be reasons to use it on main.

e.g., this would still be allowed, as without using inert, this was one of the best ways to make a web page hidden to a screen reader when a modal dialog was open.

<body>
  <header aria-hidden=true>...</header>
  <main aria-hidden=true>...</main>
  <div role=dialog ...>...</div>
</body>

so we wouldn't want to fail a page for doing something valid like that. the html/body aria-hidden=true is far more harmful and i would submit is far more common due to misunderstanding how to use aria, rather than being done for a valid use case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Allowed aria-* attributes
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nschonni @scottaohara