w3c / aria

Accessible Rich Internet Applications (WAI-ARIA)
https://w3c.github.io/aria/
Other
639 stars 123 forks source link

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

Open rahimabdi opened 10 months ago

rahimabdi commented 10 months ago

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

pkra commented 10 months ago

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

pkra commented 10 months ago

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 commented 10 months ago

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