Skip to content

Incorrect name and no state is defined for the ‘Hamburger’ menu button. #2308

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
msftedad opened this issue Nov 28, 2023 · 2 comments
Closed

Comments

@msftedad
Copy link

Bug Report

Steps to reproduce

  1. Open the URL: PWABuilder - Packaging for the Google Play Store in browser.
  2. Turn on the narrator through Ctrl+ Windows+ Enter key.
  3. Navigate to the hamburger menu through tab key.
  4. Observe the issue.

Current behaviour

Incorrect name and no state are defined for the ‘Hamburger’ menu button, Screen reader is only announcing as ‘Menu button.

Attachments:

Incorrect name and no state is defined for the ‘Hamburger’ menu button (2)
Incorrect.name.and.no.state.is.defined.for.the.Hamburger.menu.button.3.1.mp4

Expected behaviour

Proper name and state should be defined for the ‘Hamburger’ menu button.Screen reader should announce as ‘Hamburger menu’ button 'collapsed’ when it is in collapsed state and expanded when it is in expanded state.

Other relevant information

  • Docsify version: 4.4.4
  • [ X] Bug still occurs when all/other plugins are disabled?

  • Docsify plugins (if the bug happens when plugins enabled, please try to isolate the issue): NA

Please create a reproducible sandbox

(https://codesandbox.io/static/img/play-codesandbox.svg)](https://codesandbox.io/s/307qqv236)](https://docs.pwabuilder.com/#/)

Mention the docsify version in which this bug was not present (if any)

NA

@paulhibbitts
Copy link
Collaborator

paulhibbitts commented Nov 29, 2023

Thanks very much for reporting this issue @msftedad . Re: un-merged PR #2304 might it still be possible to consider the status 'collapsed' and 'expanded' to the aria-labels for the hamburger menu element @jhildenbiddle ?

Perhaps another option instead of using a UI element name (which might not be familiar to all users) something more action-related like "Expand sidebar, button" and "Collapse sidebar, button" might be label options to consider?

@jhildenbiddle
Copy link
Member

Fixed via #2254.

These updates will be available in the next release. Until then, you can verify the changes by visiting the develop branch preview.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants