Merge pull request #742 from mattico/narrow-device-style-fixes

Reduce the margins on header icons on narrow devices
rust-lang · Jul 25, 2018 · a287a0d · a287a0d
2 parents b4e15e5 + e7afb33
commit a287a0d
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 4 deletions.
diff --git a/src/theme/book.css b/src/theme/book.css
@@ -217,7 +217,7 @@ table thead td {
 #menu-bar i,
 #menu-bar .icon-button {
   position: relative;
-  margin: 0 10px;
+  margin: 0 8px;
   z-index: 10;
   line-height: 50px;
   cursor: pointer;
@@ -227,6 +227,12 @@ table thead td {
   -ms-transition: color 0.5s;
   transition: color 0.5s;
 }
+@media only screen and (max-width: 420px) {
+  #menu-bar i,
+  #menu-bar .icon-button {
+    margin: 0 5px;
+  }
+}
 #menu-bar #print-button {
   margin: 0 15px;
 }
@@ -237,6 +243,9 @@ html:not(.sidebar-visible) #menu-bar:not(:hover).folded > #menu-bar-sticky-conta
   -ms-transform: translateY(-60px);
   transform: translateY(-60px);
 }
+.left-buttons {
+  margin: 0 5px;
+}
 .no-js .left-buttons {
   display: none;
 }

diff --git a/src/theme/stylus/menu.styl b/src/theme/stylus/menu.styl
@@ -14,7 +14,10 @@
 
     i, .icon-button {
         position: relative
-        margin: 0 10px
+        margin: 0 8px
+        @media only screen and (max-width: $narrow-device-max-width) {
+            margin: 0 5px
+        }
         z-index: 10
         line-height: 50px
         cursor: pointer
@@ -30,8 +33,9 @@ html:not(.sidebar-visible) #menu-bar:not(:hover).folded > #menu-bar-sticky-conta
     transform: translateY(-60px);
 }
 
-.no-js .left-buttons {
-    display: none
+.left-buttons {
+    .no-js & { display: none }
+    margin: 0 5px
 }
 
 .menu-title {

diff --git a/src/theme/stylus/variables.styl b/src/theme/stylus/variables.styl
@@ -4,3 +4,4 @@ $content-max-width = 750px
 $content-min-width = 320px
 $page-plus-sidebar-width = $content-max-width + $sidebar-width + $page-padding * 2
 $sidebar-reflow-width = $sidebar-width + $content-min-width
+$narrow-device-max-width = 420px