Skip to content

Commit

Permalink
Wrap sidebar navigation in <nav> sectioning element
Browse files Browse the repository at this point in the history
This will provide semantic context to the unordered navigation list.

Ref: All page content must be contained by landmarks: https://dequeuniversity.com/rules/axe/4.1/region
  • Loading branch information
barrymcgee committed Jan 8, 2021
1 parent b46f447 commit 08ac2a2
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions includes/sidebar.html
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
<a href="/{{ currentLanguage }}" class="h4-mktg text-white no-underline no-wrap pl-2 flex-auto">{% data ui.header.github_docs %}</a>
</div>

<nav>
{% if error == '404' or page.relativePath == 'index.md' %}
<ul class="sidebar-products mt-4">
{% include sidebar-homepage %}
Expand All @@ -17,4 +18,5 @@
{% include sidebar-specific-product %}
</ul>
{% endif %}
</nav>
</div>

0 comments on commit 08ac2a2

Please sign in to comment.