search

vadikom / smartmenus

Advanced jQuery website menu plugin. Mobile first, responsive and accessible list-based website menus that work on all devices.
http://www.smartmenus.org
MIT License
593 stars 164 forks source link

[A11Y] aria-current on active menu item #239

Open laryn opened 11 months ago

laryn commented 11 months ago

Description of the bug

An accessibility review on a site in development called out the following:

No aria dictating the "active" element in the navigation for a non-visual user using a screen reader

The solution?

aria-current="page" on the active <a>

-- Cross-posted: https://github.com/backdrop/backdrop-issues/issues/6234

vadikom commented 11 months ago

Yep, that should do it. This is fixed in the upcoming v2.0.