-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The accessible menu test is incorrect #20
Comments
I took that from the theme review documentation here: https://make.wordpress.org/themes/handbook/review/accessibility/required/#keyboard-navigation
|
This is why I tried to express that checking the menu is not that simple. If opening sub menus on hover was required, then that would exclude any theme where you open submenus by clicking a button. I want to check the tests thoroughly and with help from the accessibility team, but I will not be able to do that until after the 5.8 FSE MVP dates. |
We can cycle back on these tests seeing that they are considered We can focus on the |
The menu check is incorrect; opening a sub menu on hover must not be required.
Hovering is the least accessible way.
It should not even be mentioned as recommended.
The text was updated successfully, but these errors were encountered: