Fixes jekyll#80, jekyll#106. Make the hamburger work with pure CSS. (j…

…ekyll#111) Merge pull request 111
mika-cn · Apr 7, 2017 · cdbde06 · cdbde06
1 parent 9e0accc
commit cdbde06
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 9 deletions.
diff --git a/_includes/header.html b/_includes/header.html
@@ -7,13 +7,16 @@
 
     {% if page_paths %}
       <nav class="site-nav">
-        <span class="menu-icon">
-          <svg viewBox="0 0 18 15" width="18px" height="15px">
-            <path fill="#424242" d="M18,1.484c0,0.82-0.665,1.484-1.484,1.484H1.484C0.665,2.969,0,2.304,0,1.484l0,0C0,0.665,0.665,0,1.484,0 h15.031C17.335,0,18,0.665,18,1.484L18,1.484z"/>
-            <path fill="#424242" d="M18,7.516C18,8.335,17.335,9,16.516,9H1.484C0.665,9,0,8.335,0,7.516l0,0c0-0.82,0.665-1.484,1.484-1.484 h15.031C17.335,6.031,18,6.696,18,7.516L18,7.516z"/>
-            <path fill="#424242" d="M18,13.516C18,14.335,17.335,15,16.516,15H1.484C0.665,15,0,14.335,0,13.516l0,0 c0-0.82,0.665-1.484,1.484-1.484h15.031C17.335,12.031,18,12.696,18,13.516L18,13.516z"/>
-          </svg>
-        </span>
+        <input type="checkbox" id="nav-trigger" class="nav-trigger" />
+        <label for="nav-trigger">
+          <span class="menu-icon">
+            <svg viewBox="0 0 18 15" width="18px" height="15px">
+              <path fill="#424242" d="M18,1.484c0,0.82-0.665,1.484-1.484,1.484H1.484C0.665,2.969,0,2.304,0,1.484l0,0C0,0.665,0.665,0,1.484,0 h15.031C17.335,0,18,0.665,18,1.484L18,1.484z"/>
+              <path fill="#424242" d="M18,7.516C18,8.335,17.335,9,16.516,9H1.484C0.665,9,0,8.335,0,7.516l0,0c0-0.82,0.665-1.484,1.484-1.484 h15.031C17.335,6.031,18,6.696,18,7.516L18,7.516z"/>
+              <path fill="#424242" d="M18,13.516C18,14.335,17.335,15,16.516,15H1.484C0.665,15,0,14.335,0,13.516l0,0 c0-0.82,0.665-1.484,1.484-1.484h15.031C17.335,12.031,18,12.696,18,13.516L18,13.516z"/>
+            </svg>
+          </span>
+        </label>
 
         <div class="trigger">
           {% for path in page_paths %}

diff --git a/_sass/minima/_layout.scss b/_sass/minima/_layout.scss
@@ -28,6 +28,10 @@
   float: right;
   line-height: 56px;
 
+  .nav-trigger {
+      display: none;
+  }
+
   .menu-icon {
     display: none;
   }
@@ -51,6 +55,15 @@
     border-radius: 5px;
     text-align: right;
 
+    label[for="nav-trigger"] {
+      display: block;
+      float: right;
+      width: 36px;
+      height: 36px;
+      z-index: 2;
+      cursor: pointer;
+    }
+
     .menu-icon {
       display: block;
       float: right;
@@ -65,12 +78,12 @@
       }
     }
 
-    .trigger {
+    input ~ .trigger {
       clear: both;
       display: none;
     }
 
-    &:hover .trigger {
+    input:checked ~ .trigger {
       display: block;
       padding-bottom: 5px;
     }