roles that allow aria-expanded and aria-roles.js question

[scottaohara]

ARIA 1.2 has fixed an error where ARIA 1.1 allowed aria-expanded on a number of elements that didn't make sense. e.g., role=alert aria-expanded=.... ARIA issue where this topic was discussed - fixed in 2019, so has been part of the initial and current ARIA 1.2 CR draft.

I noticed in the aria-roles.js file that aria-expanded is still marked as an allowed attribute on these roles, which at first made sense to me because the file notes on the first line that it is referencing ARIA 1.1... so since they were allowed then, if axe is still referencing ARIA 1.1 then they would still be allowed, though their use by authors is suspect / probably incorrect.

However, upon further review of the file, there are a number of roles that were introduces in ARIA 1.2 that are also included... which again also makes sense as many of them have been implemented by browsers... but this then made me wonder if it would be a good idea for axe to also correct the aria-expanded allowances, even though the correction is tied to ARIA 1.2.

Regardless of when axe might want to pull in the correction, I'd be happy to create a PR to remove the aria-expanded allowances from the necessary roles in this file.

[WilcoFiers]

Thanks for the update Scott. It's a little late for 4.4, but putting it on the list for 4.5.