Remove loads of comments

nationalarchives · Nov 21, 2023 · 345b0b2 · 345b0b2
1 parent 440bc08
commit 345b0b2
Show file tree

Hide file tree

Showing 29 changed files with 54 additions and 304 deletions.
diff --git a/.storybook/preview.js b/.storybook/preview.js
@@ -2,7 +2,6 @@ import "../src/nationalarchives/all.scss";
 import { a11yConfig } from "./storybook-config";
 import { customViewports } from "./viewports";
 import Cookies from "../src/nationalarchives/lib/cookies.mjs";
-// import isChromatic from "chromatic/isChromatic";
 
 document.documentElement.classList.add(
   "tna-template",
@@ -14,9 +13,7 @@ if (window.self !== window.top) {
 document.body.classList.add("tna-template__body");
 
 export const parameters = {
-  actions: {
-    // disable: true,
-  },
+  actions: {},
   viewport: { viewports: customViewports },
   options: { showPanel: true },
   a11y: {

diff --git a/.storybook/storybook.scss b/.storybook/storybook.scss
@@ -162,7 +162,5 @@
     padding-left: 0.5rem;
 
     @include colour.colour-background("background-tint");
-
-    // @include colour.colour-border("keyline", 1px, solid, top);
   }
 }
diff --git a/.storybook/viewports.js b/.storybook/viewports.js
@@ -1,47 +1,5 @@
 export const customViewports = {
   // ...MINIMAL_VIEWPORTS,
-  // largeMin: {
-  //   name: "Smallest large device",
-  //   styles: {
-  //     width: "1025px",
-  //     height: "800px",
-  //   },
-  // },
-  // mediumMax: {
-  //   name: "Largest medium device",
-  //   styles: {
-  //     width: "1024px",
-  //     height: "800px",
-  //   },
-  // },
-  // mediumMin: {
-  //   name: "Smallest medium device",
-  //   styles: {
-  //     width: "769px",
-  //     height: "800px",
-  //   },
-  // },
-  // smallMax: {
-  //   name: "Largest small device",
-  //   styles: {
-  //     width: "768px",
-  //     height: "800px",
-  //   },
-  // },
-  // smallMin: {
-  //   name: "Smallest small device",
-  //   styles: {
-  //     width: "481px",
-  //     height: "800px",
-  //   },
-  // },
-  // tinyMax: {
-  //   name: "Largest tiny device",
-  //   styles: {
-  //     width: "480px",
-  //     height: "800px",
-  //   },
-  // },
   medium: {
     name: "Medium device",
     styles: {

diff --git a/src/nationalarchives/all.mjs b/src/nationalarchives/all.mjs
@@ -32,17 +32,6 @@ window.addEventListener("touchstart", onFirstTouch);
 window.addEventListener("keydown", onKeyDown);
 window.addEventListener("mousedown", onMouseDown);
 
-// if (document.documentElement.classList.contains("tna-template--system-theme")) {
-//   document.documentElement.classList.add(
-//     window.matchMedia("(prefers-color-scheme: dark)").matches
-//       ? "tna-template--dark-theme"
-//       : "tna-template--light-theme",
-//   );
-//   if (window.matchMedia("(prefers-contrast: more)").matches) {
-//     document.documentElement.classList.add("tna-template--high-contrast-theme");
-//   }
-// }
-
 const initAll = (options) => {
   options = typeof options !== "undefined" ? options : {};
   const $scope =

diff --git a/src/nationalarchives/components/button/button.scss b/src/nationalarchives/components/button/button.scss
@@ -23,11 +23,6 @@
 
   @include colour.colour-border("button-background", 0.25rem);
 
-  // transition:
-  //   color 200ms,
-  //   background-color 200ms,
-  //   border-color 200ms;
-
   cursor: pointer;
 
   &,
@@ -48,11 +43,6 @@
     text-decoration: none;
 
     background: transparent;
-
-    // transition:
-    //   color 50ms,
-    //   background-color 50ms,
-    //   border-color 50ms;
   }
 
   &--plain {

diff --git a/src/nationalarchives/components/card/card.scss b/src/nationalarchives/components/card/card.scss
@@ -61,9 +61,6 @@
   }
 
   &__meta {
-    // margin: 0;
-    // padding: 0;
-
     display: flex;
     flex-wrap: wrap;
     gap: 0.5rem 1.5rem;
@@ -72,20 +69,6 @@
 
     list-style: none;
 
-    // &-item {
-    //   display: flex;
-    //   align-items: center;
-
-    //   @include colour.colour-font("font-dark");
-    //   @include typography.detail-font-small;
-
-    //   .fa-solid {
-    //     margin-right: 0.75rem;
-
-    //     @include colour.colour-font("icon-light");
-    //   }
-    // }
-
     + p {
       margin-top: 1rem;
     }

diff --git a/src/nationalarchives/components/checkboxes/checkboxes.scss b/src/nationalarchives/components/checkboxes/checkboxes.scss
@@ -5,8 +5,6 @@
 @use "../../utilities";
 
 .tna-checkboxes {
-  // margin-top: 1rem;
-
   display: flex;
   flex-direction: column;
   align-items: flex-start;
@@ -75,9 +73,6 @@
         left: 1rem;
         z-index: 2;
 
-        // @include colour.colour-border("input-foreground", 0, solid);
-        // border-width: 0 0 forms.$checkbox-checkmark-width
-        //   forms.$checkbox-checkmark-width;
         @include colour.colour-border("input-background", 0, solid);
         border-width: 0 0 forms.$checkbox-checkmark-width
           forms.$checkbox-checkmark-width;
@@ -107,10 +102,7 @@
         height: 0.375rem;
         margin-top: -0.125rem;
 
-        /* stylelint-disable-next-line number-max-precision */
-        // top: 0.46875rem;
         top: 0.75rem;
-        // left: 0.375rem;
         left: 0.75rem;
       }
     }

diff --git a/src/nationalarchives/components/date-search/date-search.scss b/src/nationalarchives/components/date-search/date-search.scss
@@ -4,7 +4,6 @@
 
 .tna-date-search {
   max-width: 100%;
-  // margin-top: 1rem;
   padding: 0 0.375rem;
 
   display: block;

diff --git a/src/nationalarchives/components/filters/filters.scss b/src/nationalarchives/components/filters/filters.scss
@@ -30,10 +30,6 @@
 
     border-radius: 99rem;
 
-    // transition:
-    //   color 200ms,
-    //   background-color 200ms;
-
     &,
     &:link,
     &:visited {
@@ -44,10 +40,6 @@
       color: #fff;
 
       background-color: #004c7e;
-
-      // transition:
-      //   color 50ms,
-      //   background-color 50ms;
     }
   }
 }
diff --git a/src/nationalarchives/components/gallery/gallery.scss b/src/nationalarchives/components/gallery/gallery.scss
@@ -16,7 +16,6 @@
 
   &__items {
     width: 100%;
-    // padding-top: 2rem;
 
     display: flex;
     flex-direction: column;
@@ -106,23 +105,13 @@
 
   @include colour.on-high-contrast {
     &__items {
-      // padding-top: 2rem;
     }
 
     &__item {
-      // padding-bottom: 2rem;
-
-      // @include colour.colour-border("keyline-dark", 1px);
-
       &-index {
-        // padding-top: 0.5rem;
       }
 
       &-description {
-        // margin-bottom: 2rem;
-        // padding: 0 1rem 1rem;
-
-        // border: none;
       }
     }
   }

diff --git a/src/nationalarchives/components/gallery/gallery.stories.js b/src/nationalarchives/components/gallery/gallery.stories.js
@@ -1,7 +1,5 @@
 import Gallery from "./template.njk";
 import macroOptions from "./macro-options.json";
-// import { expect } from "@storybook/jest";
-// import { within, userEvent } from "@storybook/testing-library";
 
 const argTypes = {
   heading: { control: "object" },
@@ -62,14 +60,14 @@ Standard.args = {
               },
             ]
           : index === 1
-          ? [
-              {
-                id: "tab-1",
-                title: "Transcript",
-                body: "<p>TEST TRANSCRIPT</p>",
-              },
-            ]
-          : null,
+            ? [
+                {
+                  id: "tab-1",
+                  title: "Transcript",
+                  body: "<p>TEST TRANSCRIPT</p>",
+                },
+              ]
+            : null,
     })),
 
   classes: "tna-gallery--demo",

diff --git a/src/nationalarchives/components/header/header.scss b/src/nationalarchives/components/header/header.scss
@@ -169,7 +169,7 @@
 
     @include colour.colour-background("font-dark");
 
-    transition: height 1ms 100ms;
+    // transition: height 1ms 100ms;
 
     &::before,
     &::after {
@@ -183,10 +183,10 @@
 
       background-color: inherit;
 
-      transition:
-        top 100ms 100ms,
-        bottom 100ms 100ms,
-        transform 100ms;
+      // transition:
+      //   top 100ms 100ms,
+      //   bottom 100ms 100ms,
+      //   transform 100ms;
 
       content: "";
     }
@@ -225,10 +225,10 @@
 
         &::before,
         &::after {
-          transition:
-            top 100ms,
-            bottom 100ms,
-            transform 100ms 100ms;
+          // transition:
+          //   top 100ms,
+          //   bottom 100ms,
+          //   transform 100ms 100ms;
         }
 
         &::before {
@@ -264,8 +264,6 @@
     justify-content: flex-end;
     gap: 0.5rem 1.5rem;
 
-    // order: 1;
-
     list-style: none;
 
     line-height: 1;
@@ -387,7 +385,6 @@
 
     &__navigation-toggle {
       &.tna-column {
-        // padding-right: gridVars.$gutter-width-tiny;
         padding-left: 0;
       }
     }

diff --git a/src/nationalarchives/components/header/header.stories.js b/src/nationalarchives/components/header/header.stories.js
@@ -210,44 +210,3 @@ Mobile.play = async ({ args, canvasElement, step }) => {
 
   $navigationToggle.blur();
 };
-
-// export const NoJavascript = Template.bind({});
-// NoJavascript.args = {
-//   navigation: [
-//     {
-//       text: "Alpha",
-//       href: "#/alpha",
-//       selected: true,
-//     },
-//     {
-//       text: "Beta",
-//       href: "#/beta",
-//     },
-//     {
-//       text: "Gamma",
-//       href: "#/gamma",
-//     },
-//   ],
-//   classes: "tna-header--demo",
-// };
-// NoJavascript.play = async ({ args, canvasElement, step }) => {
-//   document.documentElement.classList.remove("tna-template--js-enabled");
-//   canvasElement.innerHTML = canvasElement.innerHTML;
-//   const $navigationToggleWrapper = canvasElement.querySelector(
-//     `.tna-header__navigation-toggle`,
-//   );
-//   $navigationToggleWrapper.innerHTML = "";
-
-//   const canvas = within(canvasElement);
-
-//   // const $navigationItems = canvasElement.querySelector(
-//   //   `.tna-header__navigation-items`,
-//   // );
-//   // const [$linkA, $linkB, $linkC] = args.navigation.map((navigationItem) =>
-//   //   canvas.getByText(navigationItem.text),
-//   // );
-//   // await expect($navigationItems).toBeVisible();
-//   // await expect($linkA).toBeVisible();
-//   // await expect($linkB).toBeVisible();
-//   // await expect($linkC).toBeVisible();
-// };
diff --git a/src/nationalarchives/components/message/message.scss b/src/nationalarchives/components/message/message.scss
@@ -12,15 +12,10 @@
 
   color: colour.brand-colour("black");
 
-  @include colour.colour-background("keyline");
-
-  @include colour.colour-border("keyline-dark", 0.5rem, solid, left);
-
-  // &--yellow {
   background-color: colour.brand-colour("cream");
 
+  @include colour.colour-border("keyline-dark", 0.5rem, solid, left);
   border-left-color: colour.brand-colour("yellow");
-  // }
 
   &__icon {
     margin-right: 1rem;