From 40b8c32481573a8b116584a86c17c36e040c4ecb Mon Sep 17 00:00:00 2001
From: Gareth Barnard <1058419+gjb2048@users.noreply.github.com>
Date: Thu, 7 Feb 2019 13:17:43 +0000
Subject: [PATCH] Fix: Logo breaks header when using 'old navbar'.

---
 CHANGES.txt                    |  1 +
 layout/tiles/header.php        |  1 -
 less/essential/header.less     | 27 ++++++++++++++++++---------
 less/essential/navigation.less |  6 ++----
 style/essential.css            | 28 ++++++++++++++++++----------
 5 files changed, 39 insertions(+), 24 deletions(-)

diff --git a/CHANGES.txt b/CHANGES.txt
index c322d324..6dcb9848 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -5,6 +5,7 @@ New in 3.4.1.5
 - FIX: Participants menu only on course and module pages.
 - FIX: Administration / Naviagtion menu still appearing when flat navigation is on because they are not in the side-pre region.
 - FIX: 'courselistteachericon' not showing.
+- FIX: Logo breaks header when using 'old navbar'.
 - NEW: Show / hide the participants menu with the 'participantsmenu' setting in the 'Header' settings.
 - NEW: Separate 'People' from 'This course' menu and create 'Participants' menu.
 - NEW: Course content search attribute and order.
diff --git a/layout/tiles/header.php b/layout/tiles/header.php
index 0d234661..27467350 100755
--- 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
index bf090902..b3748075 100644
--- 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
index 4ff8afa9..a93d17b8 100644
--- 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
index ab440893..412d8c48 100644
--- a/style/essential.css
+++ b/style/essential.css
@@ -23054,10 +23054,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;
@@ -23089,16 +23085,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;
@@ -23272,11 +23280,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;
   }
 }