Merge pull request #77 from nhsuk/sidebar-spacing

Side nav spacing + accessibility improvements
nhsuk · May 9, 2024 · 909bce3 · 909bce3
2 parents 5d8e489 + 874bfa3
commit 909bce3
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 11 deletions.
diff --git a/docs/_includes/layouts/component.njk b/docs/_includes/layouts/component.njk
@@ -4,16 +4,8 @@
 
 {% set sidebar %}
   {{ appSideNavigation({
-    classes: 'nhsuk-u-padding-top-6',
     currentPath: page.url,
-    sections: [
-      {
-        heading: {
-          text: "Components"
-        },
-        items: collections.component
-      }
-    ]
+    items: collections.component
   }) }}
 {% endset %}
 

diff --git a/docs/assets/css/_example.scss b/docs/assets/css/_example.scss
@@ -46,6 +46,7 @@
 
   a:hover {
     color: $nhsuk-link-hover-color;
+    text-decoration: none;
   }
 
   a:focus {

diff --git a/docs/assets/css/components/_side-navigation.scss b/docs/assets/css/components/_side-navigation.scss
@@ -7,12 +7,12 @@
 
   display: block;
   margin-left: - nhsuk-spacing(2);
-  padding: nhsuk-spacing(4) 0 0;
+  padding: nhsuk-spacing(6) 0 0;
 }
 
 .app-side-navigation__title {
   @include nhsuk-font($size: 19);
-  color: $color_nhsuk-grey-2;
+  color: $color_nhsuk-grey-1;
   font-weight: normal;
   margin: 0;
   padding: nhsuk-spacing(2);
@@ -40,6 +40,7 @@
 
   a:hover {
     color: $nhsuk-link-hover-color;
+    text-decoration: underline;
   }
 
   a:focus {
@@ -79,6 +80,9 @@
 }
 
 .app-side-navigation--mobile {
+  .app-side-navigation {
+    padding: 0 0 nhsuk-spacing(4);
+  }
   @include mq($from: desktop) {
     display: none;
   }