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.

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

Missing role=presentation for "Name from prohibited" #2073

Open
rahimabdi opened this issue Nov 4, 2023 · 3 comments
Open

Missing role=presentation for "Name from prohibited" #2073

rahimabdi opened this issue Nov 4, 2023 · 3 comments

Comments

@rahimabdi
Copy link
Contributor

rahimabdi commented Nov 4, 2023

Describe your concern

While reviewing 5.2.8.6 Roles which cannot be named (Name prohibited), I noticed that the role none is in the list but not presentation. I assumed that since they are synonyms, that listing one or the other is OK however, the 5.3.3 Document Structure Roles section lists both none/presentation.

My concern is that a quick glance of section 5.2.8.6 may yield a false impression that the presentation role can be named, and that it's not a true alias for none.

I tried to add the presentation role in section 5.2.8.6 as a simple <li> however, it looks like the list is dynamically generated by aria/common/script/aria.js using the characteristics table for each role's definition in aria/index.html (and the "Name From: " row and whether it is "prohibited"); this appears to be why presentation is missing from 5.2.8.6 because the presentation role definition has no characteristics table unlike the none role definition.

Should presentation/none always be listed together (like in 5.3.3)? And should the presentation role definition have the same characteristics table as none?

Link to the version of the specification or documentation you were looking at at.

Link to documentation: https://w3c.github.io/aria/

Does the issue exists in the editors draft (the editors draft is the most recent draft of the specification)?
Yes

@cookiecrook cookiecrook added the editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo label Nov 9, 2023
@pkra pkra added Agenda-Editors and removed editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo labels Nov 9, 2023
@pkra
Copy link
Member

pkra commented Nov 9, 2023

Note from the call: check other synonyms (img/image) as well. Related: #2076

@pkra
Copy link
Member

pkra commented Nov 10, 2023

As Rahim already indicated, we need to enable aria.js (in aria-common) to pick up on synonyms.

Would it be enough to look up synonyms and add their links parenthetically? E.g., "image (synonyms: img)" or some better wording along those lines.

@pkra
Copy link
Member

pkra commented Nov 10, 2023

This should probably transfer to aria-common but it might be more sensible to preview the changes in a PR here first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants