Changes header navigation to a <nav> element, style by ID

django · Jun 1, 2023 · ee8a5cc · ee8a5cc
1 parent 1b2823b
commit ee8a5cc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/djangoproject/scss/_style.scss b/djangoproject/scss/_style.scss
@@ -575,7 +575,7 @@ blockquote {
         }
     }
 
-    [role="navigation"] {
+    #header-navigation {
         background: $green-dark;
 
         width: 100%;

diff --git a/djangoproject/templates/includes/header.html b/djangoproject/templates/includes/header.html
@@ -5,7 +5,7 @@
     <div class="mobile-toggle">
       {% include "includes/toggle_theme.html" %}
     </div>
-    <div role="navigation">
+    <nav id="top-navigation">
       <ul>
         <li{% if 'start' in request.path %} class="active"{% endif %}>
           <a href="{% url 'overview' %}">Overview</a>
@@ -38,6 +38,6 @@
           {% include "includes/toggle_theme.html" %}
         </li>
       </ul>
-    </div>
+    </nav>
   </div>
 </div>