- Issue created by @jldust
- πΊπΈUnited States jldust
This is just a first pass at adding accessible menus into the 4.x Prototype branch. It is ready for community feedback & additional testing.
- Status changed to Needs review
over 1 year ago 11:15pm 24 March 2023 - @jldust opened merge request.
- πΊπΈUnited States michaelablackham
This looks great!
The only change I am looking for is to clean it up a little bit:
- Add
aria-label
to the<nav>
- Remove the
aria-labelledby
on the<nav>
- Remove the visually hidden
h2
.
This way it is just a little cleaner:
<nav aria-label="main" id="block-mainnavigation">
- Add
-
jldust β
committed ac036994 on 4.x
Issue #3345126 by jldust, michaelablackham: Mobile Menu Support
-
jldust β
committed ac036994 on 4.x
- Status changed to Fixed
over 1 year ago 1:22am 5 April 2023 - πΊπΈUnited States jldust
Thanks @michaelablackham for your feedback and testing! This has been merged into the 4.x branch and will be included in the next release.
- Status changed to Fixed
over 1 year ago 3:49am 5 April 2023