Issue 5635 - Implement website-default navbar component

static/scss/_navbar.scss

 // This file is part of help.eyeo.com.
 // Copyright (C) 2017 Eyeo GmbH
 //
 // help.eyeo.com is free software: you can redistribute it and/or modify
 // it under the terms of the GNU General Public License as published by
 // the Free Software Foundation, either version 3 of the License, or
 // (at your option) any later version.
 //
 // help.eyeo.com is distributed in the hope that it will be useful,
 // but WITHOUT ANY WARRANTY; without even the implied warranty of
 // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 // GNU General Public License for more details.
 //
 // You should have received a copy of the GNU General Public License
 // along with help.eyeo.com.  If not, see <http://www.gnu.org/licenses/>.
 
 /*******************************************************************************
- * Navbar component (Table Layout Method)
+ * Navbar component
  ******************************************************************************/
 
-#table-navbar .navbar .container
-{
-  display: table;
-  width: 100%;
-  max-width: $large-desktop-width;
-}
-
-#table-navbar .navbar button

[juliandoucette, 2017/11/06 12:36:43]

NIT: Perhaps button should normally inherit color?

[ire, 2017/11/08 17:14:46]

Done.

-{
-  color: inherit;
-}
-
-#table-navbar .navbar-branding,
-#table-navbar .toggle-navbar-collapse,
-#table-navbar .navbar-collapse
-{
-  display: table-cell;
-  padding-top: $small-space;
-  padding-bottom: $small-space;
-  vertical-align: middle;
-}
-
-/* .navbar .toggle-navbar-collapse
-****************************************************************************/
-
-#table-navbar .toggle-navbar-collapse
-{
-  display: none;
-}
-
-.js #table-navbar .toggle-navbar-collapse
-{
-  display: table-cell;
-
-  @media (min-width: $tablet-breakpoint)
-  {
-    display: none;
-  }
-}
-
-#table-navbar .toggle-navbar-collapse button

[juliandoucette, 2017/11/06 12:36:43]

TOL: Perhaps we could use (extend) a (link button | unstyled button) instead.

[ire, 2017/11/08 17:14:46]

Done.

-{
-  min-width: 100px;
-  border: 0;
-  background-color: transparent;
-  cursor: pointer;
-}
-
-.js #table-navbar .toggle-navbar-collapse .close-label
-{
-  display: none;
-}
-
-#table-navbar .expanded .toggle-navbar-collapse .open-label
-{
-  display: none;
-}
-
-#table-navbar .expanded .toggle-navbar-collapse .close-label
-{
-  display: inline;
-}
-
-/* .navbar .navbar-collapse
-****************************************************************************/
-
-#table-navbar .navbar-collapse
-{
-  display: table-row;
-  text-align: left;
-
-  @media (min-width: $tablet-breakpoint)
-  {
-    display: table-cell;
-    text-align: right;
-  }
-}
-
-[dir="rtl"] #table-navbar .navbar-collapse
-{
-  text-align: right;
-
-  @media (min-width: $tablet-breakpoint)
-  {
-    text-align: left;
-  }
-}
-
-.js #table-navbar .navbar-collapse
-{
-  display: none;
-
-  @media (min-width: $tablet-breakpoint)
-  {
-    display: table-cell;
-  }
-}
-
-#table-navbar .expanded .navbar-collapse
-{
-  display: table-row;
-}
-
-.no-js #table-navbar .navbar-collapse:after,
-#table-navbar .expanded .navbar-collapse:after
-{
-  @media (max-width: $tablet-breakpoint + 1px)
-  {
-    display: block;
-    height: $small-space;
-    content: "";
-  }
-}
-
-
-/*******************************************************************************
- * Navbar component (Float Method)
- ******************************************************************************/
-
-#floated-navbar .navbar
+.navbar
 {
   @extend .clearfix;
 }
 
-#floated-navbar .navbar button
+/* .navbar .navbar-container
+ ****************************************************************************/
+
+.navbar-container
 {
-  border: 0;
-  color: inherit;
-  background-color: transparent;
-  cursor: pointer;
+  @if ($add-horizontal-navbar-padding)
+  {
+    @extend .container;
+  }
+  @else
+  {
+    @include container($container-width, 0px);
+  }
 }
 
-#floated-navbar .navbar-branding,
-#floated-navbar .toggle-navbar-collapse,
-#floated-navbar .navbar-collapse
+/* .navbar .navbar-branding
+ ****************************************************************************/
+
+.navbar-branding
 {
-  padding-top: $small-space;
-  padding-bottom: $small-space;
+  padding-top: $navbar-padding + ($navbar-padding / 4);
+  padding-bottom: $navbar-padding + ($navbar-padding / 4);
+  line-height: 1;
+
+  @if ($add-horizontal-navbar-padding != true)
+  {
+    padding-right: $navbar-padding;
+    padding-left: $navbar-padding;
+  }
+}
+
+.navbar-branding img
+{
+  height: $navbar-branding-height;
+  vertical-align: text-top;
 }
 
 /* .navbar .toggle-navbar-collapse
-  ****************************************************************************/
+ ****************************************************************************/
 
-#floated-navbar .toggle-navbar-collapse
+.toggle-navbar-collapse
 {
   display: none;
-  border: 0;
-  background-color: transparent;
-  cursor: pointer;
+  padding-top: ($navbar-branding-height / 2) + ($navbar-padding / 2);
+  padding-bottom: ($navbar-branding-height / 2) + ($navbar-padding / 2);
+
+  @if ($add-horizontal-navbar-padding != true)
+  {
+    padding-right: $navbar-padding;
+    padding-left: $navbar-padding;
+  }
 }
 
-.js #floated-navbar .toggle-navbar-collapse
+.js .toggle-navbar-collapse
 {
   display: block;
 
   @media (min-width: $tablet-breakpoint)
   {
     display: none;
   }
 }
 
-.js #floated-navbar .toggle-navbar-collapse .close-label
-{
-  display: none;
-}
+/* .navbar .navbar-collapse
+ ****************************************************************************/
 
-#floated-navbar .expanded .toggle-navbar-collapse .open-label
-{
-  display: none;
-}
-
-#floated-navbar .expanded .toggle-navbar-collapse .close-label
-{
-  display: inline;
-}
-
-/* .navbar .navbar-collapse
-  ****************************************************************************/
-
-#floated-navbar .navbar-collapse
+.navbar-collapse
 {
   clear: both;
-  padding-top: 0;
 
   @media (min-width: $tablet-breakpoint)
   {
     float: right;
     clear: none;
-    padding-top: $small-space;
   }
 }
 
-[dir="rtl"] #floated-navbar .navbar-collapse
+[dir="rtl"] .navbar-collapse
 {
   float: left;
 }
 
-.js #floated-navbar .navbar-collapse
+.js .navbar-collapse
 {
   display: none;
 
   @media (min-width: $tablet-breakpoint)
   {
     display: block;
   }
 }
 
-#floated-navbar .expanded .navbar-collapse
+.expanded .navbar-collapse
 {
   display: block;
+  width: 100%;
+
+  @media (min-width: $tablet-breakpoint)
+  {
+    width: auto;
+  }
 }
+
+/* .navbar .navbar-nav
+ ****************************************************************************/
+
+@if ($add-horizontal-navbar-padding)
+{
+  .navbar .navbar-nav
+  {
+    padding-bottom: $navbar-padding / 2;
+
+    @media (min-width: $tablet-breakpoint)
+    {
+      margin-right: -($navbar-padding / 2);
+      margin-left: -($navbar-padding / 2);
+      padding-bottom: 0;
+    }
+  }
+}
+
+.navbar .navbar-nav li
+{
+  @media (min-width: $tablet-breakpoint)
+  {
+    display: inline-block;
+  }
+}
+
+.navbar .navbar-nav a
+{
+  display: block;
+  padding-top: $navbar-padding / 2;
+  padding-bottom: $navbar-padding / 2;
+
+  @if ($add-horizontal-navbar-padding != true)
+  {
+    padding-right: $navbar-padding;
+    padding-left: $navbar-padding;
+  }
+
+  @media (min-width: $tablet-breakpoint)
+  {
+    padding-top: ($navbar-branding-height / 2) + ($navbar-padding / 2);
+    padding-right: $navbar-padding / 2;
+    padding-bottom: ($navbar-branding-height / 2) + ($navbar-padding / 2);
+    padding-left: $navbar-padding / 2;
+  }
+}