Fix: Logo breaks header when using 'old navbar'.

gjb2048 · Feb 7, 2019 · b8c6332 · b8c6332
1 parent c153c73
commit b8c6332
Show file tree

Hide file tree

Showing 5 changed files with 42 additions and 24 deletions.
diff --git a/CHANGES.txt b/CHANGES.txt
@@ -1,6 +1,7 @@
 New in 3.6.0.4
 ==============
 - FIX: 'courselistteachericon' not showing.
+- FIX: Logo breaks header when using 'old navbar'.
 - NEW: Course content search attribute and order.
 
 New in 3.6.0.3

diff --git a/layout/tiles/header.php b/layout/tiles/header.php
@@ -58,7 +58,6 @@
     <div id="page-header" class="clearfix<?php echo ($oldnavbar) ? ' oldnavbar' : ''; echo ($haslogo) ? ' logo' : ' nologo'; ?>">
         <div class="container-fluid">
             <div class="row-fluid">
-                <!-- HEADER: LOGO AREA -->
 <?php
 if (!$haslogo) {
     echo '<div class="pull-left">';

diff --git a/less/essential/header.less b/less/essential/header.less
@@ -93,11 +93,6 @@
         }
     }
 
-    a.logo {
-        background-position: left center;
-        float: left;
-    }
-
     div.titlearea {
         margin-left: 80px;
         padding-left: 10px;
@@ -140,9 +135,16 @@
 }
 
 #page-header {
-    &.logo {
-        min-height: 52px;
+    &.logo .logo-container {
+        height: 52px;
+        .logo {
+            line-height: 52px;
+            img {
+                max-height: 52px;
+            }
+        }
     }
+
     &.nologo {
         height: 52px;
         max-height: 52px;
@@ -151,9 +153,16 @@
 
 @media (min-width: @navbarCollapseDesktopWidth) {
     #page-header {
-        &.logo {
-            min-height: 64px;
+        &.logo .logo-container {
+            height: 64px;
+            .logo {
+                line-height: 64px;
+                img {
+                    max-height: 64px;
+                }
+            }
         }
+
         &.nologo {
             height: 64px;
             max-height: 64px;

diff --git a/less/essential/navigation.less b/less/essential/navigation.less
@@ -34,14 +34,12 @@
 
 #essentialnavbar {
     &.oldnavbar {
-        &.nologo {
-            top: 52px;
-        }
+        top: 52px;
     }
 }
 
 @media (min-width: @navbarCollapseDesktopWidth) {
-    #essentialnavbar.oldnavbar.nologo {
+    #essentialnavbar.oldnavbar {
         top: 64px;
     }
 }

diff --git a/style/essential.css b/style/essential.css
@@ -20104,6 +20104,9 @@ ul.indented-list {
 .dir-ltr .dir-ltr-hide {
   display: none;
 }
+.font-weight-bold {
+  font-weight: 700 !important;
+}
 .embed-responsive {
   position: relative;
   display: block;
@@ -23466,10 +23469,6 @@ table#greyboxleft h2 {
 #page-header #essentialicons button {
   margin: 0 0 0 5px;
 }
-#page-header a.logo {
-  background-position: left center;
-  float: left;
-}
 #page-header div.titlearea {
   margin-left: 80px;
   padding-left: 10px;
@@ -23501,16 +23500,28 @@ table#greyboxleft h2 {
     max-height: 80px;
   }
 }
-#page-header.logo {
-  min-height: 52px;
+#page-header.logo .logo-container {
+  height: 52px;
+}
+#page-header.logo .logo-container .logo {
+  line-height: 52px;
+}
+#page-header.logo .logo-container .logo img {
+  max-height: 52px;
 }
 #page-header.nologo {
   height: 52px;
   max-height: 52px;
 }
 @media (min-width: 980px) {
-  #page-header.logo {
-    min-height: 64px;
+  #page-header.logo .logo-container {
+    height: 64px;
+  }
+  #page-header.logo .logo-container .logo {
+    line-height: 64px;
+  }
+  #page-header.logo .logo-container .logo img {
+    max-height: 64px;
   }
   #page-header.nologo {
     height: 64px;
@@ -23684,11 +23695,11 @@ ul.socials li {
     margin-top: 0;
   }
 }
-#essentialnavbar.oldnavbar.nologo {
+#essentialnavbar.oldnavbar {
   top: 52px;
 }
 @media (min-width: 980px) {
-  #essentialnavbar.oldnavbar.nologo {
+  #essentialnavbar.oldnavbar {
     top: 64px;
   }
 }