Fixed Mobile nav trigger does not have an accessible name

[36degrees]

Affected version

v2.2.2

The mobile nav trigger (p-nav-menuTrigger) does not have an accessible name. Although there is a nested <span class="p-nav-menuText">Menu</span> this is hidden using display: none and so not available to assistive technologies. It may make sense to use an aria-label instead.

It's also currently an <a> tag despite having a role="button" – given it doesn't have an href a <button> may make more sense.

(There are some more issues with actually accessing the mobile nav once you've hit the trigger if you're using a screen reader… I'll try and write them up as a separate issue when I get time.)

 

[XF Bug Bot]

Thank you for reporting this issue, it has now been resolved. We are aiming to include any changes that have been made in a future XF release (2.2.3).

Change log:

Ensure the mobile navigation trigger has an accessible name

There may be a delay before changes are rolled out to the XenForo Community.