-
Notifications
You must be signed in to change notification settings - Fork 877
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
Shortcode Breadcrumbs not marked up as a nav and decorative elements not hidden from screen reader users #20680
Comments
Thank you @amberhinds for reaching out. I can see the same happening for our Breadcrumbs block. There is no I'm asking internally how we can improve. |
@amberhinds thanks for your feedback. Quickly chiming in to add my 2 cents. It's a long time I don't work on the Yoast plugin code but I seem to recall we added an option to make the container a I think we should consider a couple important points:
The shortcode allows users to place the breadcrumbs wherever they want. Making ti always be a |
This is one that frustrates me big time.
Ultimately, after going back and forth with this in my mind, I think you should just add the Anyway, not going to tell you how to make your plugin, this is Yoast after all, just wanted to add some thoughts about how I approach these situations. Thanks. |
Wrote it up: nvaccess/nvda#15536 |
@alexstine thanks for your feedback and for creating the NVDA issue, interesting one. One of the things to consider is that, historically, shortcodes could be only added inside the post content. In that scenario, a nav inside a main would have been arguable in addition to trigger the nested landmarks issue you mentioned. Now that with the WordPress Site editor the shortcodes can be placed everywhere, even inside another group with a landmark element and even inside another mav, things are a bit more complex. I have no strong preferences on whether to make it a nav element by default but then there should be at least a warning with some meaningful info and recommended best practices for users. Overall, I think on the Yoast side we should check a few things Cc @jeroenrotty
|
Please give us a description of what happened
The breadcrumbs output by the shortcode is not wrapped in a labeled nav tag and the divider elements (which are decorative) are not hidden from screen readers.
To Reproduce
Step-by-step reproduction instructions
Expected results
This is the expected code:
Actual results
This is the code currently being output:
Other Notes
This request to switch to using list markup would also be a great accessibility enhancement but is not as urgent as using the proper nav tag.
#5055
Technical info
Used versions
The text was updated successfully, but these errors were encountered: