fix a11y issues

readthedocs · Nov 23, 2024 · e33ac7e · e33ac7e
1 parent 8d4d394
commit e33ac7e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/sphinx_rtd_theme/layout.html b/sphinx_rtd_theme/layout.html
@@ -117,7 +117,7 @@
           {# the master_doc variable was renamed to root_doc in Sphinx 4 (master_doc still exists in later Sphinx versions) #}
           {%- set _logo_url = logo_url|default(pathto('_static/' + (logo or ""), 1)) %}
           {%- set _root_doc = root_doc|default(master_doc) %}
-          <a href="{{ pathto(_root_doc) }}"{% if not theme_logo_only %} class="icon icon-home"{% endif %}>
+          <a href="{{ pathto(_root_doc) }}"{% if not theme_logo_only %} aria-label="Go to Homepage" class="icon icon-home"{% endif %}>
             {% if not theme_logo_only %}{{ project }}{% endif %}
             {%- if logo or logo_url %}
               <img src="{{ _logo_url }}" class="logo" alt="{{ _('Logo') }}"/>

diff --git a/sphinx_rtd_theme/static/js/theme.js b/sphinx_rtd_theme/static/js/theme.js
diff --git a/src/theme.js b/src/theme.js
@@ -114,7 +114,7 @@ function ThemeNav () {
         $('.wy-menu-vertical ul').not('.simple').siblings('a').each(function () {
             var link = $(this);
                 expand =
-                    $('<button class="toctree-expand" title="Open/close menu"></button>');
+                    $('<button class="toctree-expand" title="Open/close menu" role="presentation"></button>');
             expand.on('click', function (ev) {
                 self.toggleCurrent(link);
                 ev.stopPropagation();