From a53ea1ff93022ec991d392b53c18b4fb5fb2297d Mon Sep 17 00:00:00 2001
From: kevwhitt-hee <kevin.whittaker@hee.nhs.uk>
Date: Wed, 12 Feb 2025 09:09:32 +0000
Subject: [PATCH 1/3] moves skip link outside of header for consistency with
 nhsuk

---
 DigitalLearningSolutions.Web/Views/Shared/_Layout.cshtml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/DigitalLearningSolutions.Web/Views/Shared/_Layout.cshtml b/DigitalLearningSolutions.Web/Views/Shared/_Layout.cshtml
index 648ee41a51..bd3e9469fe 100644
--- a/DigitalLearningSolutions.Web/Views/Shared/_Layout.cshtml
+++ b/DigitalLearningSolutions.Web/Views/Shared/_Layout.cshtml
@@ -65,11 +65,11 @@
     {
         @await Html.PartialAsync("~/Views/Shared/_GoogleTagManagerBodyTagJs.cshtml")
     }
+    <script>document.body.className = ((document.body.className) ? document.body.className + ' js-enabled' : 'js-enabled');</script>
+    <a class="nhsuk-skip-link" href="#maincontent">Skip to main content</a>
     <div id="pagewrapper">
         <header class="nhsuk-header nhsuk-header--organisation nhsuk-header--white" role="banner">
-            <script>document.body.className = ((document.body.className) ? document.body.className + ' js-enabled' : 'js-enabled');</script>
             <partial name="_CookieConsentPartial" />
-            <a class="nhsuk-skip-link" href="#maincontent">Skip to main content</a>
             @if (Configuration["ShowAlertBanner"] == "True")
             {
                 <div class="dls-alert-banner no-print">

From a5560ba1bf7abb7e4cc4c38cbb62915740cd0861 Mon Sep 17 00:00:00 2001
From: kevwhitt-hee <kevin.whittaker@hee.nhs.uk>
Date: Wed, 12 Feb 2025 09:20:48 +0000
Subject: [PATCH 2/3] Removes active nav bottom border indicator

---
 DigitalLearningSolutions.Web/Styles/layout.scss | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/DigitalLearningSolutions.Web/Styles/layout.scss b/DigitalLearningSolutions.Web/Styles/layout.scss
index 6ec87cccb2..07305d9ab9 100644
--- a/DigitalLearningSolutions.Web/Styles/layout.scss
+++ b/DigitalLearningSolutions.Web/Styles/layout.scss
@@ -32,14 +32,8 @@ footer {
 }
 
 .nhsuk-header__navigation-item.selected {
-  @include mq($from: large-desktop) {
-    box-shadow: inset 0 -4px 0 #d8dde0;
-  }
-
-  @include mq($until: large-desktop) {
-    a {
-      font-weight: bold;
-    }
+  a {
+    font-weight: bold;
   }
 }
 

From 42a4e7dcd5e6399117e64dee5c73a6f6958e0b80 Mon Sep 17 00:00:00 2001
From: kevwhitt-hee <kevin.whittaker@hee.nhs.uk>
Date: Thu, 13 Feb 2025 08:49:27 +0000
Subject: [PATCH 3/3] TD-5346 Fixes mobile header service name styling using
 overrides

---
 .../Styles/layout.scss                        | 24 +++++++++++++++++++
 .../Views/Shared/_Layout.cshtml               |  1 -
 2 files changed, 24 insertions(+), 1 deletion(-)

diff --git a/DigitalLearningSolutions.Web/Styles/layout.scss b/DigitalLearningSolutions.Web/Styles/layout.scss
index 07305d9ab9..f815adafe4 100644
--- a/DigitalLearningSolutions.Web/Styles/layout.scss
+++ b/DigitalLearningSolutions.Web/Styles/layout.scss
@@ -37,6 +37,30 @@ footer {
   }
 }
 
+.nhsuk-header__logo {
+  @include mq($until: large-desktop) {
+    max-width: unset;
+
+    .nhsuk-header__link--service {
+      align-items: center;
+      display: flex;
+      -ms-flex-align: center;
+      margin-bottom: 0;
+      width: auto;
+    }
+
+    .nhsuk-header__service-name {
+      padding-left: nhsuk-spacing(3);
+      max-width: unset;
+    }
+  }
+}
+.nhsuk-navigation-container {
+  @include mq($until: tablet) {
+    margin-top: unset;
+  }
+}
+
 body:not(.js-enabled) {
   .nhsuk-header__menu {
     display: none;
diff --git a/DigitalLearningSolutions.Web/Views/Shared/_Layout.cshtml b/DigitalLearningSolutions.Web/Views/Shared/_Layout.cshtml
index bd3e9469fe..68d57ea940 100644
--- a/DigitalLearningSolutions.Web/Views/Shared/_Layout.cshtml
+++ b/DigitalLearningSolutions.Web/Views/Shared/_Layout.cshtml
@@ -153,7 +153,6 @@
             @{
                 var visualSeparatorNoNavBar = (bool?)ViewData[LayoutViewDataKeys.DoNotDisplayNavBar] == true ? "visual-separator-no-nav-bar" : string.Empty;
             }
-            <div class="visual-separator @visualSeparatorNoNavBar"></div>
 
             <feature name="@(FeatureFlags.UserFeedbackBar)">
                 <partial name="_UserFeedbackBarPartial" model=@(userResearchUrl) />