Improve aria markup to satisfy WAVE accessibility tests
Closed this issue · 8 comments
At present the use of role="menubar" in the views is invalid unless you (at least) also state role="menuitem" for each nav link within the bar.
Probably wants a bit more consideration than just that, to make sure these really are at all useful under aria.
Needs changes in Bootstrap4TopNavWithDropdowns and possibly a few others
_LoginPartialWithDropdowns (main cs Core repo)
Bs4ChildDropdown
@JimKerslake could we resolve this one soon? I think role="menuitem" needs to be added to each link. This make a typical cloudscribe horizontal menu pass tests in WAVE, II've found.
cloudscribe.web.navigation up to v6.0.1
but also need to chase this up the chain so it gets used by the template...
cloudscribe.Web.SiteMap.FromNavigation.6.0.1
cloudscribe.Core.Web 6.0.9
Several more views needing the same, it turns out
cloudscribe.Web.Navigation up to 6.0.2
cloudscribe.Web.SiteMap.FromNavigation up to 6.0.2
Going to need to push a new cloudscribe.Core.Web to deliver those...
cloudscribe.Core.Web. 6.0.21
According to WAVE there are no issues with the nav or dropdowns as things stand.