roots / sage

WordPress starter theme with Laravel Blade components and templates, Tailwind CSS, and a modern development workflow
https://roots.io/sage/
MIT License
12.71k stars 3.06k forks source link

Bug: additional `<nav>` elements in partials need accessible names #3105

Closed knowler closed 8 months ago

knowler commented 1 year ago

Terms

Description

What's wrong?

The following <nav> elements need accessible names (i.e. can be set with aria-label or aria-labelledby).

Possible solutions

Use aria-label to set accessible names for these regions. Do not use the word “navigation” in the name (it’s redundant).

References

Steps To Reproduce

  1. This is the current state of the templates.

Expected Behavior

When there’s more than one navigation region on a page, they should have accessible names to distinguish them from each other.

Actual Behavior

These unnamed navigation regions might appear more prominent than they should be.

Relevant Log Output

No response

Versions

HEAD