Add banner role to logo container in sidebar header

All page content must be contained by landmarks: https://dequeuniversity.com/rules/axe/4.1/region
shahrokh90 · Jan 8, 2021 · b46f447 · b46f447
1 parent d4d7017
commit b46f447
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/includes/header.html b/includes/header.html
@@ -5,7 +5,7 @@
 
   <header class="container-xl px-3 px-md-6 pt-3 pb-2 position-relative d-flex flex-justify-between width-full {% if error == '404' %} d-md-none {% endif %}">
 
-    <div class="d-flex flex-items-center d-lg-none" style="z-index: 3;" id="github-logo-mobile">
+    <div class="d-flex flex-items-center d-lg-none" style="z-index: 3;" id="github-logo-mobile" role="banner">
       <a href="/{{ currentLanguage }}" aria-hidden="true" tabindex="-1">
         {% octicon "mark-github" height="32" class="text-black" %}
       </a>

diff --git a/includes/sidebar.html b/includes/sidebar.html
@@ -1,7 +1,7 @@
 <!-- product > category > maptopic > article -->
 <div class="sidebar d-none d-lg-block">
 
-  <div class="d-flex flex-items-center p-4 position-sticky top-0 sidebar-background-color" style="z-index: 3;" id="github-logo">
+  <div class="d-flex flex-items-center p-4 position-sticky top-0 sidebar-background-color" style="z-index: 3;" id="github-logo" role="banner">
     <a href="/{{ currentLanguage }}" class="text-white" aria-hidden="true" tabindex="-1">
       {% octicon "mark-github" height="32" %}
     </a>