🐛 BUG: fixing sidebar overlap on narrow screens

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

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/sphinx_book_theme/layout.html b/sphinx_book_theme/layout.html
@@ -25,8 +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">
-    {% if not is_single_page %}
+<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 %}
 </div>

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