Spacing tweaks (#25)

* Adjust spacing on hero mobile view

* Remove vertical spacing on <hr>

* Update CHANGELOG.md

* Add tna-button-group around button in footer

* Fix <hr> double border

CHANGELOG.md

@@ -22,6 +22,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Links in footer and text in buttons have balanced wrapping applied
 - Header and header group class modifiers have been moved to separate blocks (`tna-heading tna-heading--xl` -> `tna-heading-xl`)
 - `space-below` has been replaced with `space-above` and all paddings in between page elements reassessed
+- Tweak spacing on hero component and `<hr>` elements
+- Added `tna-button-group` around newletter subscription button in the footer
 - Updated Storybook to `7.5.1`
 
 ### Deprecated

src/nationalarchives/components/footer/template.njk

@@ -78,10 +78,12 @@
         <p>
           Subscribe for regular news, updates and priority booking for events.
         </p>
-        <a href="#" class="tna-button tna-button--accent" role="button">
-          <i class="fa-solid fa-envelope"></i>
-          Subscribe
-        </a>
+        <div class="tna-button-group">
+          <a href="#" class="tna-button tna-button--accent" role="button">
+            <i class="fa-solid fa-envelope"></i>
+            Subscribe
+          </a>
+        </div>
       </div>
     </div>
     {%- endif -%}

src/nationalarchives/components/hero/hero.scss

@@ -246,7 +246,7 @@
     }
 
     &__content-inner {
-      padding: 1.5rem 0;
+      padding: 2rem 0;
 
       background-color: transparent;
     }

src/nationalarchives/utilities/_global.scss

@@ -135,7 +135,7 @@ canvas {
 }
 
 hr {
-  margin: 2rem 0;
+  margin: 0;
 
   border-width: 1px 0 0;
   @include colour.colour-border("keyline");