Clarify case-insensitive treatment of ARIA roles - #2838
Conversation
✅ Deploy Preview for wai-aria ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
🚀 Deployed on https://deploy-preview-2838--wai-aria.netlify.app |
| </p> | ||
| <p> | ||
| User agents MUST use the first token in the sequence of tokens in the <a><code>role</code> attribute</a> value that matches the name of any non-abstract | ||
| User agents MUST use the first token in the sequence of tokens in the <a><code>role</code> attribute</a> value that is an [=ASCII case-insensitive=] match for the name of any non-abstract |
There was a problem hiding this comment.
I think there needs to be a matching change in the definition of the role attribute (Section 8.1) which has
The first name literal of a non-abstract WAI-ARIA role in the list of tokens in the role attribute defines the role according to which the user agent MUST process the element.
Come to think of it, I think it would be better to remove this sentence here and only clarify this in the definition of the role attribute / Section 8.1.
There was a problem hiding this comment.
Agreed @pkra, could it be specified in both places? I've updated the PR.
There was a problem hiding this comment.
We just merged #2798 and section 8.1 is now the definitive place.
I strongly prefer not to duplicate normative statements - they get out of sync. Especially here, where there is little reason to look for a duplication all the way in Section 4 should we change something in Section 8.1.
HaTheo
left a comment
HaTheo
left a comment
There was a problem hiding this comment.
LGTM. I am not too familiar with the repercussions of defining it both in 4.1 and 8.1, but I saw pkra seemed to prefer defining it only in 8.1.
| </li> | ||
| <li> | ||
| The first name literal of a non-abstract <abbr title="Accessible Rich Internet Applications">WAI-ARIA</abbr> role in the list of tokens in the <a><code>role</code> attribute</a> defines | ||
| The first [=ASCII case-insensitive=] name literal of a non-abstract <abbr title="Accessible Rich Internet Applications">WAI-ARIA</abbr> role in the list of tokens in the <a><code>role</code> attribute</a> defines |
There was a problem hiding this comment.
I think this sentence should be replaced with the one above; "name literal" is just odd here.
4 participants
🚀 Netlify Preview:
🔄 this PR updates the following sspecs:
Closes #2548
This PR updates the "WAI-ARIA Roles" section specifying treatment of ARIA roles should be case-insensitive (see WPT tests below).