🐛 BUG: fixing sidebar overlap on narrow screens

executablebooks · Aug 8, 2020 · ee3b945 · ee3b945
1 parent 6439b66
commit ee3b945
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/sphinx_book_theme/layout.html b/sphinx_book_theme/layout.html
@@ -25,7 +25,7 @@
 <!-- Docs TOC is "d-none d-xl-block col-xl-2" -->
 
 {% block docs_sidebar %}
-<div class="col-12 {{ sidebar_width_class }} bd-sidebar site-navigation show" id="site-navigation">
+<div class="col-12 {{ sidebar_width_class }} bd-sidebar site-navigation show{% if is_single_page %} single-page{% endif %}" id="site-navigation">
     {% if not is_single_page %}
         {% include "sidebar.html" %}
     {%- endif %}

diff --git a/sphinx_book_theme/static/sphinx-book-theme.scss b/sphinx_book_theme/static/sphinx-book-theme.scss
@@ -685,6 +685,11 @@ button.topbarbtn img {
         &.collapsing {
             height: 0px !important;
         }
+
+        // If we're only a single page, sidebar should hide at narrow widths.
+        &.single-page {
+            display: none;
+        }
     }
 }