Fix button typeface and remove :link selectors

nationalarchives · Jul 25, 2024 · bda73ad · bda73ad
1 parent 62deee7
commit bda73ad
Show file tree

Hide file tree

Showing 11 changed files with 5 additions and 24 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,9 +14,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Deprecated
 ### Removed
+
+- Removed styling for links with the `:link` pseudo class - all `<a>` elements should posess an `href` attribute by default
+
 ### Fixed
 
 - Further fixed gallery component layout when CSS loads but JS fails
+- Buttons using the `<button>` element now have the correct typeface (they were using the browser default)
 
 ### Security
 

diff --git a/src/nationalarchives/components/breadcrumbs/breadcrumbs.scss b/src/nationalarchives/components/breadcrumbs/breadcrumbs.scss
@@ -50,7 +50,6 @@
     display: inline-block;
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("font-base");
     }
@@ -94,7 +93,6 @@
   @include colour.on-high-contrast {
     &__link {
       &,
-      &:link,
       &:visited {
         @include colour.colour-font("link");
       }

diff --git a/src/nationalarchives/components/button/button.scss b/src/nationalarchives/components/button/button.scss
@@ -23,7 +23,7 @@ $button-border-width: 4px !default;
   cursor: pointer;
 
   @include typography.font-size(18);
-  @include typography.main-font-weight-bold;
+  @include typography.main-font(true);
 
   @include colour.colour-background("button-background");
 
@@ -33,7 +33,6 @@ $button-border-width: 4px !default;
   print-color-adjust: exact;
 
   &,
-  &:link,
   &:visited {
     @include colour.colour-font("button-text");
   }
@@ -69,7 +68,6 @@ $button-border-width: 4px !default;
     border-color: transparent;
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("link");
     }
@@ -90,7 +88,6 @@ $button-border-width: 4px !default;
     @include colour.colour-border("button-accented-background");
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("button-accented-text");
     }
@@ -192,7 +189,6 @@ $button-border-width: 4px !default;
     @include colour.colour-border("button-background");
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("button-text");
     }

diff --git a/src/nationalarchives/components/error-summary/error-summary.scss b/src/nationalarchives/components/error-summary/error-summary.scss
@@ -30,7 +30,6 @@
     @include typography.main-font-weight-bold;
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("form-error-text");
     }

diff --git a/src/nationalarchives/components/global-header/global-header.scss b/src/nationalarchives/components/global-header/global-header.scss
@@ -46,7 +46,6 @@
 
     &--link {
       &,
-      &:link,
       &:visited {
         @include colour.colour-font("font-base");
       }
@@ -185,7 +184,6 @@
     @include colour.colour-border("keyline", 0.5px, solid, bottom);
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("font-base");
     }
@@ -217,7 +215,6 @@
     text-decoration: none;
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("font-light");
     }

diff --git a/src/nationalarchives/components/header/header.scss b/src/nationalarchives/components/header/header.scss
@@ -40,7 +40,6 @@
     @include typography.font-size(16);
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("font-base");
     }
@@ -74,7 +73,6 @@
 
     &--link {
       &,
-      &:link,
       &:visited {
         color: inherit;
       }
@@ -220,15 +218,13 @@
     text-decoration: none;
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("font-light");
     }
 
     &:hover,
     &--selected {
       &,
-      &:link,
       &:visited {
         @include colour.colour-font("font-base");
         @include typography.interacted-text-decoration;
@@ -282,7 +278,6 @@
     }
 
     &,
-    &:link,
     &:visited {
       color: inherit;
     }
@@ -362,7 +357,6 @@
 
     &__top-navigation-item-link {
       &,
-      &:link,
       &:visited {
         @include colour.colour-font("font-light");
       }

diff --git a/src/nationalarchives/components/index-grid/index-grid.scss b/src/nationalarchives/components/index-grid/index-grid.scss
@@ -64,7 +64,6 @@
 
     .tna-index-grid__item:hover & {
       &,
-      &:link,
       &:visited {
         @include typography.interacted-text-decoration;
       }

diff --git a/src/nationalarchives/components/quick-filters/quick-filters.scss b/src/nationalarchives/components/quick-filters/quick-filters.scss
@@ -46,7 +46,6 @@
     @include colour.colour-border("keyline", 1px);
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("font-base");
     }
@@ -58,7 +57,6 @@
 
   &__item--selected &__link {
     &,
-    &:link,
     &:visited,
     &:hover {
       @include colour.accent;

diff --git a/src/nationalarchives/components/skip-link/skip-link.scss b/src/nationalarchives/components/skip-link/skip-link.scss
@@ -25,7 +25,6 @@
   clip-path: inset(50%) !important;
 
   &,
-  &:link,
   &:visited {
     @include colour.colour-font("button-text", $important: true);
   }

diff --git a/src/nationalarchives/components/tabs/tabs.scss b/src/nationalarchives/components/tabs/tabs.scss
@@ -43,7 +43,6 @@
     @include typography.heading-font;
 
     &,
-    &:link,
     &:visited {
       @include colour.colour-font("font-dark");
     }

diff --git a/src/nationalarchives/global-header-package.scss b/src/nationalarchives/global-header-package.scss
@@ -160,7 +160,6 @@ hr {
 
 .tna-button {
   &,
-  &:link,
   &:visited {
     text-decoration: none;
   }
@@ -182,7 +181,6 @@ hr {
 
       &-link {
         &,
-        &:link,
         &:visited {
           display: block;
         }
-Original file line number
+Diff line change
@@ Expand Up / @@ -30,7 +30,6 @@ @@
         @include typography.main-font-weight-bold;
         &,
-        &:link,
         &:visited {
           @include colour.colour-font("form-error-text");
         }
@@ Expand Down @@