Simplify styles for the menu button

Using `currentcolor` and `color: inherit` is IMHO more expressive (we're
saying we want to use the same color as the text) and makes it easier to
customize these colors in other CONSUL installations. We also remove
duplication as we can use the same styles for both the admin and the
public layouts.

app/assets/stylesheets/admin.scss

@@ -159,14 +159,6 @@ $table-header:   #ecf1f6;
     right: 12px;
   }
 
-  .menu-icon {
-
-    &::after {
-      background: #000;
-      box-shadow: 0 7px 0 #000, 0 14px 0 #000;
-    }
-  }
-
   .notifications.unread-notifications::after {
     color: $admin-color;
   }

app/assets/stylesheets/layout.scss

@@ -668,15 +668,8 @@ body > header,
 }
 
 .menu-icon {
-
+  @include hamburger($color: currentcolor, $color-hover: currentcolor);
-  &.dark {
+  color: inherit;
-
-    &:hover::after,
-    &::after {
-      background: #fff;
-      box-shadow: 0 7px 0 #fff, 0 14px 0 #fff;
-    }
-  }
 }
 
 .title-bar {

app/views/layouts/_header.html.erb

@@ -16,7 +16,7 @@
     <div class="top-bar">
 
       <span data-responsive-toggle="responsive-menu" data-hide-for="medium" class="float-right">
-        <button type="button" class="menu-icon dark" data-toggle></button>
+        <button type="button" class="menu-icon" data-toggle></button>
         <%= t("application.menu") %>
       </span>