Merge pull request #809 from sinsong/master

Fix .bd-sizebar-secondary and .secondary-toggle breakpoint
executablebooks · Jun 12, 2024 · bd60e86 · bd60e86
2 parents 3b6fd68 + 9939b7a
commit bd60e86
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/sphinx_book_theme/assets/styles/sections/_header-article.scss b/src/sphinx_book_theme/assets/styles/sections/_header-article.scss
@@ -26,7 +26,7 @@ button.sidebar-toggle.primary-toggle {
 
 // Secondary toggle mimics behavior of "persistent header" div of PST
 button.sidebar-toggle.secondary-toggle {
-  @media (min-width: $breakpoint-lg) {
+  @media (min-width: $breakpoint-xl) {
     display: none;
   }
   @media (max-width: $breakpoint-md) {

diff --git a/src/sphinx_book_theme/assets/styles/sections/_sidebar-secondary.scss b/src/sphinx_book_theme/assets/styles/sections/_sidebar-secondary.scss
@@ -6,7 +6,7 @@
   // So that it sticks to the top of the page instead of the header height
   top: 0;
 
-  @media (max-width: $breakpoint-lg) {
+  @media (max-width: $breakpoint-xl) {
     // So that tooltips don't overlap
     z-index: $zindex-tooltip + 1;
   }
@@ -36,7 +36,7 @@
   }
 
   // Wide screen behavior
-  @media (min-width: $breakpoint-lg) {
+  @media (min-width: $breakpoint-xl) {
     background: var(--pst-color-background);
     height: fit-content;
     transition: max-height 0.4s ease;