Issue 2675 - Implemented responsive header for web.eyeo.com

static/css/styles.css

 /*
  * Fonts
  */
 @font-face {
   font-family: "Lato";
   font-style: normal;
   font-weight: 400;
   src: local("Lato Regular"), local("Lato-Regular"), url(/fonts/Lato-Regular.woff) format('woff');
 }
 @font-face {
@@ skipping 83 matching lines @@
   font-size:14px;
 }
 
 h6
 {
   font-size:12px;
 }
 
 #header
 {
-  display: block;
-  position: absolute;
-  background-color: #ffffff;
+  position: fixed;
+  background-color: #fff;
   width: 100%;
   top: 0;

[saroyanm, 2016/06/22 07:54:19]

Detail: CSS number values should specify units where possible
While you are changing this selector styles.

[juliandoucette, 2016/06/23 17:25:41]

Done.

   left: 0;
   box-shadow: 0 0 12px rgba(0,0,0, 0.09);
   z-index: 10000;
-}
-
-#header.fixed
-{
-  position: fixed;
-}
-
-#header-content
-{
-  display: table;
-  width: 100%;
+  text-align: right;
+}
+
+#header .content-block
+{
+  position: relative;
+  padding: 15px 10px;
+  min-height: 32px;
+}
+
+#header.top .content-block
+{
+  padding: 25px 10px;
 }
 
 #logo
 {
-  display: table-cell;
-  padding: 13px 0;
-}
-
-#header.fixed #logo
-{
-  padding: 5px 0;
+  position: absolute;
+  top: 7px;
+  left: 10px;
+}
+
+#header.top #logo
+{
+  top: 17.5px;

[saroyanm, 2016/06/22 07:54:19]

17.5px, just curious why are you using 0.5px accuracy for positioning ?

[juliandoucette, 2016/06/23 17:25:41]

Changed.

+}
+
+#header .menu-button
+{
+  margin: 0 0 0 25px;

[saroyanm, 2016/06/22 07:54:19]

Detail: CSS number values should specify units where possible

[juliandoucette, 2016/06/23 17:25:41]

Done.

 }
 
 #menu
 {
-  display: table-cell;
+  display: inline;
   vertical-align: middle;
-  width: 100%;
 }
 
 .assistive-text
 {
   position: absolute;
+  top: -99999px;

[saroyanm, 2016/06/22 07:54:18]

I'm not really familiar with this implementation, can you please describe why we
are positioning this element absolutely and why it's not possible to change the
visibility ?

[juliandoucette, 2016/06/23 17:25:41]

See:

https://make.wordpress.org/accessibility/2015/02/09/hiding-text-for-screen-re...

It affects the shape of the block when focused.

+  left: -99999px;
   clip: rect(1px, 1px, 1px, 1px);
 }
 
 .menu-list
 {
+  display: inline;
+  margin: 0px;
+  font-size: 15px;
   list-style: none;
-  margin: 0;
-  padding: 0;
-  min-height: 36px;
-  font-size: 14px;
-  float: right;
 }
 
 .menu-list li
 {
-  position: relative;
+  display: inline;
+  font-weight: bold;
   white-space: nowrap;
-  display: block;
-  font-size: 15px;
-  font-weight: bold;
-  line-height: 36px;
-  outline-width: 0;
-  float: left;
-  margin-right: 1px;
 }
 
 .menu-list li a
 {
   color: #979797;
   padding: 0 10px;
-
-  -webkit-transition: all 0.3s ease;
-  -moz-transition: all 0.3s ease;
-  -o-transition: all 0.3s ease;
-  transition: all 0.3s ease;
 }
 
 .menu-list li:hover > a,
 .menu-list li.current-menu-item > a
 {
   color: #373737;
 }
 
-.menu-button
-{
-  float: right;
-  margin-left: 25px;
+#header-hamburger
+{
+  display: none;
+}
+
+#header-hamburger:hover .icon-bar,
+#header-hamburger:focus .icon-bar,
+#header-hamburger:active .icon-bar
+{
+  background-color: #000;
+}
+
+
+.icon-bar
+{
+  display: block;
+  width: 22px;
+  height: 2px;
+  border-radius: 1px;
+  background-color: #888;
+}
+
+.icon-bar+.icon-bar
+{
+  margin-top: 4px;
 }
 
 #content:before,
 #content:after,
-#header:before,
-#header:after,
 .columns-container:before,
 .columns-container:after
 {
   content: " ";
   display: table;
 }
 
 #content:after,
-#header:after,
 .columns-container:after
 {
   clear: both;
 }
 
 #footer
 {
   color: #666;
   background-color: #252525;
   padding-top: 30px;
@@ skipping 560 matching lines @@
   background-color: #747474;
 }
 
 .why-work-here-button:hover,
 .why-work-here-button:focus
 {
   background-color: #333;
 }
 
 
-@media (max-width: 750px)
-{
+@media (max-width: 768px)
+{
+  body
+  {
+    padding-top: 50px;
+  }
+
+  #header .content-block,
+  #header.top .content-block
+  {
+    padding: 9px 10px;
+  }
+
+  #header.top #logo
+  {
+    top: 7px;
+  }
+
+  #logo > img
+  {
+    height: 34px;
+  }
+
+  #menu
+  {
+    display: none;
+  }
+
+  #menu.open
+  {
+    display: block;
+  }
+
+  #header-hamburger
+  {
+    position: absolute;
+    top: 9px;
+    right: 10px;
+    display: block;
+    padding: 0px;
+    margin: 0px 5px;
+    background: none;
+  }
+
+  .menu-list
+  {
+    margin-top: 45px;
+    padding: 0px;
+    display: block;
+  }
+
+  .menu-list > li
+  {
+    display: block;
+    border-top: 1px solid #eee;
+  }
+
+  .menu-list > li > a
+  {
+    display: block;
+    margin: 0px;
+    padding: 10px;
+    font-size: 14px
+  }
+
+  .menu-button
+  {
+    position: absolute;
+    top: 9px;
+    right: 50px;
+    height: 32px;
+    padding: 0 10px;
+    line-height: 32px;
+    font-size: 14px;
+  }
+
   .columns-container .column
   {
     width: 100%;
     margin-left: 0;
     clear: both;
   }
 
-  #logo
-  {
-    padding: 5px 0;
-  }
-
-  #menu .menu-list
-  {
-    clear: both;
-    margin-left: -150px;
-  }
-
-  #menu .menu-button
-  {
-    margin-top: 5px;
-  }
-
   #press-info .sidebar-left
   {
     text-align: inherit;
   }
 
   #releases li > *
   {
     display: block;
   }
 
@@ skipping 25 matching lines @@
     padding-right: .1em;
     border-right-width: 1px;
     border-right-width: 1px;
   }
 
   #panels-openings
   {
     margin-top: -40px;
   }
 }