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

static/css/styles.css

Index: static/css/styles.css
diff --git a/static/css/styles.css b/static/css/styles.css
index 58dd31179abe67d3a0bebc6fb1fbd2a6cccc2567..72094bfa18af1627744297dd6461edbde9ef06ba 100644
--- a/static/css/styles.css
+++ b/static/css/styles.css
@@ -102,7 +102,7 @@ h6
 #header
 {
   display: block;
-  position: absolute;
+  position: fixed;
   background-color: #ffffff;
   width: 100%;
   top: 0;
@@ -111,33 +111,21 @@ h6
   z-index: 10000;
 }
 
-#header.fixed
+#header.top 
 {
-  position: fixed;
-}
-
-#header-content
-{
-  display: table;
-  width: 100%;
+  padding: 8px 0;
 }
 
 #logo
 {
-  display: table-cell;
-  padding: 13px 0;
-}
-
-#header.fixed #logo
-{
-  padding: 5px 0;
+  float: left;

[saroyanm, 2016/04/08 16:56:09]

Why do we need to use float ?

[juliandoucette, 2016/04/25 19:08:55]

Because I don't want it to overlap.

- absolute/fixed would overlap
- inline block would require a hacky container/separator
- floats stack when they run into each other

[juliandoucette, 2016/04/25 19:13:54]

Clarification: I'm talking about overlapping when the screen gets too small to
contain both elements side by side.

[saroyanm, 2016/05/06 14:49:27]

I think you can alternatively have logo positioned absolutely and have left
margin equal to the size of the logo so you will not have overlap. Ex.:

<logo positioned absolute width=100px></logo>
<nav width=100% text-align=right>
  <inline block margin-left=100px></inline block>
  <inline block></inline block>
  <inline block></inline block>
</nav>

What you think ?

[juliandoucette, 2016/05/11 18:11:06]

It would be better to set a minimum width on the container to prevent
overlapping. 

Absolute positioning is great for positioning blocks in absolute positions.

Floating is great for positioning blocks inline left or right.

I intended to position these blocks inline inside the header left and right.

[saroyanm, 2016/05/19 07:43:39]

You are right setting min-width is looks to be a better solution, currently we
are using bunch of floats and it's bit chaotic now and you are right absolute
positioning is also not a good solution for this case, but I would argue that
Float are not meant for layouting and they only needs to be used when there is
no alternative for layouting IMO if we can avoid floating that would be great.
"The float CSS property specifies that an element should be taken from the
normal flow and placed along the left or right side of its container, where text
and inline elements will wrap around it"
So while it's possible to achieve result without using float that will be great
and if we can use floats only for the reason they are meant for.

+  margin: 5px 0;
 }
 
 #menu
 {
-  display: table-cell;
+  float: right;
   vertical-align: middle;
-  width: 100%;
 }
 
 .assistive-text
@@ -150,8 +138,7 @@ h6
 {
   list-style: none;
   margin: 0;
-  padding: 0;
-  min-height: 36px;
+  padding: 22px 0;
   font-size: 14px;
   float: right;
 }
@@ -163,7 +150,6 @@ h6
   display: block;
   font-size: 15px;
   font-weight: bold;
-  line-height: 36px;
   outline-width: 0;
   float: left;
   margin-right: 1px;
@@ -173,11 +159,6 @@ h6
 {
   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,
@@ -189,7 +170,33 @@ h6
 .menu-button
 {
   float: right;
-  margin-left: 25px;
+  margin: 14px 0 14px 25px;
+}
+
+#header-hamberger
+{
+  display: none;
+}
+
+#header-hamberger:hover,
+#header-hamberger:active,
+#header-hamberger:focus
+{
+  background-color: transparent;
+  border: none;
+  outline: none;
+}
+
+.icon-bar {
+  background-color: #888;
+  display: block;
+  width: 22px;
+  height: 2px;
+  border-radius: 1px;
+}
+
+.icon-bar+.icon-bar {
+  margin-top: 4px;
 }
 
 #content:before,
@@ -785,29 +792,39 @@ header > *, .content-block
 }
 
 
-@media (max-width: 750px)
+@media (max-width: 768px)
 {
-  .columns-container .column
+  body
   {
-    width: 100%;
-    margin-left: 0;
-    clear: both;
+    padding-top: 64px;
   }
 
-  #logo
+  #logo > img 
   {
-    padding: 5px 0;
+    height: 34px
   }
 
-  #menu .menu-list
+  .menu-list
   {
-    clear: both;
-    margin-left: -150px;
+    padding: 12px 0;
+  }
+
+  .menu-item > a 
+  {
+    font-size: 13px
   }
 
-  #menu .menu-button
+  .menu-button
   {
-    margin-top: 5px;
+    margin: 5px 0 5px 10px;
+    font-size: 13px
+  }
+
+  .columns-container .column
+  {
+    width: 100%;
+    margin-left: 0;
+    clear: both;
   }
 
   #press-info .sidebar-left
@@ -826,6 +843,59 @@ header > *, .content-block
   }
 }
 
+@media(max-width: 660px)

[saroyanm, 2016/04/08 16:56:09]

Why do we need two media queries ? 
I think would be great to have only one, maybe we can use percentages for logo
for smaller screens or just go with small size implementation ? 
Because as much we will introduce media queries as much maintenance  effort we
will introduce for future.

[juliandoucette, 2016/04/25 19:08:55]

This media query does have an admittedly small use case... I will remove it.

+{
+  #menu
+  {
+    display: none;
+    float: none;
+  }
+
+  .menu-list
+  {
+    padding: 0;
+    float: none;
+    clear: both;
+    display: block;
+  }
+
+  #menu.open
+  {
+    display: block;
+  }
+
+  .menu-list > li
+  {
+    float: none;
+    display: block;
+    border-top: 1px solid #eee;
+  }
+
+  .menu-list > li > a
+  {
+    padding: 10px ;
+    display: block;
+    margin: 0;
+  }
+
+  .menu-button
+  {
+    height: 32px;
+    line-height: 32px;
+    padding: 0 10px;
+    margin: 7px 0;
+  }
+
+  #header-hamberger
+  {
+    display: block;
+    float: right;
+    margin: 5px -5px 0 5px;
+    background: none;
+    padding: 0 10px;
+  }
+}
+
 @media (min-width: 751px)
 {
   .ui-tabs-nav-vertical