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 86 matching lines @@
 h6
 {
   font-size:12px;
 }
 
 #header
 {
   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);
+  top: 0px;
+  left: 0px;
+  box-shadow: 0px 0px 12px rgba(0,0,0, 0.09);
   z-index: 10000;
   text-align: right;
 }
 
 #header .content-block
 {
   position: relative;
   padding: 15px 10px;
   min-height: 32px;
 }
 
 #header.top .content-block
 {
   padding: 25px 10px;
 }
 
 #logo
 {
   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.

+  top: 17px;
 }
 
 #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.

+  margin: 0px 0px 0px 25px;
 }
 
 #menu
 {
   display: inline;
   vertical-align: middle;
 }
 
 .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;
+  position: absolute !important;
+  height: 1px;
+  width: 1px;
+  overflow: hidden;
   clip: rect(1px, 1px, 1px, 1px);
 }
 
 .menu-list
 {
   display: inline;
   margin: 0px;
   font-size: 15px;
   list-style: none;
 }
@@ skipping 755 matching lines @@
     padding-right: .1em;
     border-right-width: 1px;
     border-right-width: 1px;
   }
 
   #panels-openings
   {
     margin-top: -40px;
   }
 }