search

w3c / html-aria

ARIA in HTML
https://w3c.github.io/html-aria/
Other
181 stars 49 forks source link

add allowed roles for nav element #211

Closed scottaohara closed 4 years ago

scottaohara commented 4 years ago

closes #185

as noted in the linked issue, nav element was missing allowed ARIA roles.

Preview | Diff

scottaohara commented 4 years ago

Looking at this as an exercise of progressive enhancement as well as some parity with what other elements that are exposed as landmarks allow. e.g. what might a nav be an appropriate fallback for if a more complex ARIA widget were not able to render? And if header can allow for group, for example, then might not nav?

That said, as with other issues I filed around the same time, I'm happy to trim this down if need be. But there should be some allowed roles as there were none listed, previously.

stevefaulkner commented 4 years ago

@scottaohara build failed when attempting to merge

scottaohara commented 4 years ago

@marcoscaceres could we get some help with what exactly the problem is with this?