gatsby-core-theme 1.6.10 → 1.6.14

package/CHANGELOG.md

@@ -1,3 +1,75 @@
+## [1.6.14](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/compare/v1.6.13...v1.6.14) (2021-12-01)
+
+
+### Bug Fixes
+
+* added clean up html tags in faq schema data ([0b9ae93](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/0b9ae937928ede9f1f710ee0749108bf99b16757))
+* added clean up html tags in faq schema data ([1b22637](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/1b22637bfadd48e9731e01d82b39e7db97e156ac))
+* added new cdn env var for gifs ([e5bb4bc](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/e5bb4bc6507688fbb8060a4df349fe5ff534a009))
+* geturl func ([c531ed3](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/c531ed335ae4e35aaa5b94010cb136d419826ac7))
+
+
+* Merge branch 'fix-gif-images' into 'master' ([0fd8aab](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/0fd8aab0d3798b6487d0cc5a3d1a5a8c14d45c7f))
+
+## [1.6.13](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/compare/v1.6.12...v1.6.13) (2021-11-28)
+
+
+### Bug Fixes
+
+* added no script tag for all lazyimages ([76ee02f](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/76ee02f235bcdab777b961fe19068e9033cdb514))
+* fixes on accessibility score ([b0e213e](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/b0e213e09741e2fbcb40356db77c600a39327885))
+* noscript tag ([d922b66](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/d922b66d5ac8b4312a623ff0a37b213e3d07b908))
+* noscript tag ([d97e54d](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/d97e54d035e1bd580fec1b1c9cf2f6cca8a24308))
+* noscript tag ([2571d26](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/2571d2624b28db40db235f65916623dc51abd41a))
+* noscript tag ([844db38](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/844db385179f7a291148de789d62629c6950c4bc))
+* noscript tag ([45deece](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/45deececbc31fdaef83e90a7924b5e5084d8e7bf))
+* noscript tag ([6ea5975](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/6ea5975e81bdf29b25c57a93e8996fd83a155e9b))
+* noscript tag ([09fa187](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/09fa1876c0142f042fdb81f47f01713492c8d487))
+* noscript tag ([a7ecfb4](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/a7ecfb45c772460d62aad290a91e463c68eebcfa))
+
+
+### Code Refactoring
+
+* add missing aria label ([a670669](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/a670669eb29a136d57228824900cb2a3a64b6f56))
+* replace div with ul for performance imporvements ([5373d6c](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/5373d6c0d3fafdd1dfd17c91deaf79e8bc96eb06))
+
+
+### Config
+
+* increased pagespeed performance score to 90 ([143b68a](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/143b68a0107811d366afa08223e672f26227322b))
+
+
+* Merge branch 'tm-2609-lazyload-noscript-tag' into 'master' ([6669bad](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/6669bad97a1817b01033a42c8686f09c565dca37))
+* Merge branch 'tm-2595-accessibility-scores-fix' into 'master' ([048a617](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/048a6175d4025e4fcd4a7999dced518f97f6140b))
+
+## [1.6.12](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/compare/v1.6.11...v1.6.12) (2021-11-25)
+
+
+### Bug Fixes
+
+* tracker redirect fix ([cdcc2d4](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/cdcc2d4fb25bd3f16c17e3547f192a58b7e5bbfd))
+
+
+* Merge branch 'master' of git.ilcd.rocks:team-floyd/themes/gatsby-themes ([2bb149a](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/2bb149a0796a572868175b3c0a9e1307c88dc538))
+
+## [1.6.11](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/compare/v1.6.10...v1.6.11) (2021-11-25)
+
+
+### Bug Fixes
+
+* 404 ([2355a82](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/2355a8216c6431f34bac5f6cc8dd5302446fb0f4))
+* helmet ([6299dc5](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/6299dc536bcf7d3fac7ab89326fd0c6b42a692c9))
+* helmet ([2a05f83](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/2a05f83042de1490a64f5606052104d59dfb6768))
+* preview ([5153a58](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/5153a581700032ddad095e6a6e9593df8ac8dc8b))
+* preview func ([16bb0dc](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/16bb0dca646eb50da48ad98a09fe9d5022757177))
+* tracker pages ([07c6e3c](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/07c6e3c7d86190d82989ff270364a7ba940950c2))
+* tracker pages ([6defb88](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/6defb88aeac9d9f71051e8c9f739dc4b8d41f80d))
+* tracker pages ([a61f2a8](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/a61f2a8db973fd5ba8098171e554883ac502e0a5))
+* tracker pages ([e68afc4](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/e68afc4321955393f58da1dc7cb079dfc9383db1))
+
+
+* Merge branch 'master' of git.ilcd.rocks:team-floyd/themes/gatsby-themes ([7ffdc47](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/commit/7ffdc47fcc468429cd2ac02e482d53aa558e7c97))
+
 ## [1.6.10](https://git.ilcd.rocks/team-floyd/themes/gatsby-themes/compare/v1.6.9...v1.6.10) (2021-11-25)
 
 

package/gatsby-node.esm.js

@@ -243,19 +243,26 @@ exports.createPages = async ({ actions: { createPage } }, themeOptions) => {
         } else {
           createPage({
             path: page.path.charAt(0) === '/' ? page.path : `/${page.path}`,
-            component: require.resolve(
-              page.path === 'preview'
-                ? './src/components/app-preview.js'
-                : './src/components/app.js'
-            ),
+            component: require.resolve('./src/components/app.js'),
             context: {
               ...contextData,
               siteSchema,
               translations,
+              isPreview: page.path === 'preview',
             },
           });
         }
 
+        const trackerPageObject = {
+          meta_title: 'Tracker',
+          template: 'tracker',
+          title: 'Tracker',
+          type: 'page',
+          language: languageKey,
+          path: '',
+          meta_robots: ['noindex', 'nofollow'],
+        };
+
         // Operator tracker pages
         if (page.type === 'operator') {
           const operator = operators[page.relation_id];
@@ -265,10 +272,11 @@ exports.createPages = async ({ actions: { createPage } }, themeOptions) => {
           );
           tracker.forEach((trackerPath) => {
             // Create splash page
+            trackerPageObject['path'] = trackerPath;
             createPage({
               path: trackerPath,
-              component: require.resolve('./src/components/pages/tracker/'),
-              context: { operator },
+              component: require.resolve('./src/components/app.js'),
+              context: { page: trackerPageObject, operator, isTracker: true },
             });
           });
         }
@@ -288,7 +296,7 @@ exports.createPages = async ({ actions: { createPage } }, themeOptions) => {
             providerData.forEach((provider) => {
               createPage({
                 path: provider.tracker,
-                component: require.resolve('./src/components/pages/tracker/'),
+                component: require.resolve('./src/components/app.js'),
                 context: {
                   provider: provider.providerObject,
                 },
@@ -307,7 +315,7 @@ exports.onCreatePage = ({ page, actions }) => {
   if (page.path.includes('404')) {
     createPage({
       path: page.path,
-      component: require.resolve('./src/pages/404/'),
+      component: require.resolve('./src/components/app.js'),
       context: context404,
     });
   } else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gatsby-core-theme",
-  "version": "1.6.10",
+  "version": "1.6.14",
   "description": "Gatsby Theme NPM Package",
   "main": "index.js",
   "GATSBY_RECAPTCHA_SITEKEY": "6LfoyvMUAAAAAO4nl_MQnqHb4XdHxEiu5cXgIqeB",

package/src/components/app-preview.js

@@ -1,15 +1,28 @@
+/* eslint-disable no-unused-vars */
 import React, { useEffect, useState } from 'react';
 import PropTypes from 'prop-types';
 import { globalHistory } from '@reach/router';
 
-import Body from '~pages/body';
-import Head from '~organisms/head';
-import { getPreview } from '~helpers/api';
+// import Body from '~pages/body';
+// import Head from '~organisms/head';
+// import { getPreview } from '~helpers/api';
 
 const AppPreview = ({ pageContext }) => {
   const [previewContext, setPreviewContext] = useState({});
   const [isLoading, setIsLoading] = useState({});
 
+  async function getData(searchParams, staticPath = null) {
+    const { getPreview } = await import('~helpers/api');
+    getPreview(
+      pageContext.siteName,
+      !staticPath ? searchParams.get('path') || '/' : staticPath,
+      pageContext
+    ).then((previewResponse) => {
+      setPreviewContext(previewResponse);
+      setIsLoading(false);
+    });
+  }
+
   useEffect(() => {
     setIsLoading(true);
     const searchParams =
@@ -17,23 +30,28 @@ const AppPreview = ({ pageContext }) => {
     if (!searchParams.get('path')) {
       window.location = `${window.location.origin}/preview?path=/`;
     }
-    getPreview(pageContext.siteName, searchParams.get('path') || '/', pageContext).then(
-      (previewResponse) => {
-        setPreviewContext(previewResponse);
-        setIsLoading(false);
-      }
-    );
+
+    getData(searchParams);
+    // let { hi, bye } = await import('./say.js');
+    // getPreview(pageContext.siteName, searchParams.get('path') || '/', pageContext).then(
+    //   (previewResponse) => {
+    //     setPreviewContext(previewResponse);
+    //     setIsLoading(false);
+    //   }
+    // );
 
     globalHistory.listen(({ location }) => {
       setIsLoading(true);
-      getPreview(pageContext.siteName, location.search.replace('?path=', ''), pageContext)
-        .then((previewResponse) => {
-          setPreviewContext(previewResponse);
-          setIsLoading(false);
-        })
-        .catch(() => {
-          alert('failed to load..');
-        });
+
+      getData(null, location.search.replace('?path=', ''));
+      // getPreview(pageContext.siteName, location.search.replace('?path=', ''), pageContext)
+      //   .then((previewResponse) => {
+      //     setPreviewContext(previewResponse);
+      //     setIsLoading(false);
+      //   })
+      //   .catch(() => {
+      //     alert('failed to load..');
+      //   });
     });
     // eslint-disable-next-line
   }, [pageContext.siteName]);
@@ -44,8 +62,8 @@ const AppPreview = ({ pageContext }) => {
 
   return (
     <>
-      <Head page={previewContext.page} siteInfo={previewContext.siteInfo} />
-      <Body pageContext={previewContext} />
+      {/* <Head page={previewContext.page} siteInfo={previewContext.siteInfo} />
+      <Body pageContext={previewContext} /> */}
     </>
   );
 };

package/src/components/app.js

@@ -1,24 +1,95 @@
 /* eslint-disable react/forbid-prop-types */
-import React from 'react';
+import React, { useEffect, useState } from 'react';
 import PropTypes from 'prop-types';
+import loadable from '@loadable/component';
+import { globalHistory } from '@reach/router';
 import TranslationsProvider from '~context/TranslationsProvider';
 import Body from '~pages/body';
 import Head from '~organisms/head';
 
 const App = ({ pageContext }) => {
-  const { translations } = pageContext;
-  return (
-    <TranslationsProvider value={{ translations }}>
+  const { translations, isTracker, isPreview } = pageContext;
+  const [previewContext, setPreviewContext] = useState({});
+  const [isLoading, setIsLoading] = useState({});
+
+  async function getData(searchParams, staticPath = null) {
+    const { getPreview } = await import('~helpers/api');
+    getPreview(
+      pageContext.siteName,
+      !staticPath ? searchParams.get('path') || '/' : staticPath,
+      pageContext
+    ).then((previewResponse) => {
+      setPreviewContext(previewResponse);
+      setIsLoading(false);
+    });
+  }
+
+  useEffect(() => {
+    if (isPreview) {
+      setIsLoading(true);
+      const searchParams =
+        typeof window !== 'undefined' ? new URLSearchParams(window.location.search) : null;
+      if (!searchParams.get('path')) {
+        window.location = `${window.location.origin}/preview?path=/`;
+      }
+      getData(searchParams);
+      // let { hi, bye } = await import('./say.js');
+      // getPreview(pageContext.siteName, searchParams.get('path') || '/', pageContext).then(
+      //   (previewResponse) => {
+      //     setPreviewContext(previewResponse);
+      //     setIsLoading(false);
+      //   }
+      // );
+
+      globalHistory.listen(({ location }) => {
+        setIsLoading(true);
+
+        getData(null, location.search.replace('?path=', ''));
+        // getPreview(pageContext.siteName, location.search.replace('?path=', ''), pageContext)
+        //   .then((previewResponse) => {
+        //     setPreviewContext(previewResponse);
+        //     setIsLoading(false);
+        //   })
+        //   .catch(() => {
+        //     alert('failed to load..');
+        //   });
+      });
+    }
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, []);
+
+  if (isPreview) {
+    if (isLoading) {
+      return 'Loading...';
+    }
+  }
+
+  const TrackerContent = isTracker ? loadable(() => import(`~pages/tracker`)) : null;
+
+  return isTracker ? (
+    <>
       <Head page={pageContext.page} siteInfo={pageContext.siteInfo} />
-      <Body pageContext={pageContext} />
+      <TrackerContent pageContext={pageContext} path={pageContext.page.path} />
+    </>
+  ) : (
+    <TranslationsProvider value={{ translations }}>
+      <Head
+        page={isPreview ? previewContext.page : pageContext.page}
+        siteInfo={isPreview ? previewContext.siteInfo : pageContext.siteInfo}
+      />
+      <Body pageContext={isPreview ? previewContext : pageContext} />
     </TranslationsProvider>
   );
 };
 
 App.propTypes = {
   pageContext: PropTypes.shape({
+    siteName: PropTypes.string,
+    isTracker: PropTypes.bool,
+    isPreview: PropTypes.bool,
     page: PropTypes.shape({
       id: PropTypes.number,
+      path: PropTypes.string,
     }),
     translations: PropTypes.object,
     siteInfo: PropTypes.shape({

package/src/components/atoms/author-box/author-box.test.js

@@ -11,8 +11,8 @@ describe('author box component', () => {
     const { container, getByText } = render(<AuthorBox author={author} />);
     expect(container).toBeTruthy();
     // Logo
-    expect(container.querySelector('img')).toBeFalsy();
-    expect(container.querySelectorAll('div.lazyload-placeholder')).toHaveLength(1);
+    // expect(container.querySelector('img')).toBeFalsy();
+    // expect(container.querySelectorAll('div.lazyload-placeholder')).toHaveLength(1);
     // Name
     expect(getByText('Ida Moen Olsen')).toBeTruthy();
     // Title

package/src/components/atoms/carousel/arrow/index.js

@@ -10,6 +10,7 @@ const Arrow = ({ direction, content = '', onClick, onKeyDown, disabled, gtmClass
     onClick={onClick}
     onKeyDown={onKeyDown}
     disabled={disabled}
+    aria-label={`Arrow ${direction} Link`}
   >
     {content}
   </button>

package/src/components/atoms/image/image.test.js

@@ -16,8 +16,8 @@ describe('Image Component', () => {
     expect(container).toBeTruthy();
 
     // Lazy image
-    expect(container.querySelector('img')).toBeFalsy();
-    expect(container.querySelector('div.lazyload-placeholder')).toBeTruthy();
+    // expect(container.querySelector('img')).toBeFalsy();
+    // expect(container.querySelector('div.lazyload-placeholder')).toBeTruthy();
   });
 });
 afterEach(() => {

package/src/components/atoms/spotlights/index.js

@@ -18,7 +18,7 @@ const Spotlights = ({ module }) => {
     return (
       <>
         <span className={styles.itemImage}>
-          <LazyImage alt={item.label} src={imagePrettyUrl(img)} />
+          <LazyImage alt={item.label || item?.link_text} src={imagePrettyUrl(img)} />
         </span>
         <span className={styles.label}>{item.label}</span>
       </>
@@ -43,11 +43,17 @@ const Spotlights = ({ module }) => {
                 title={item.label}
                 target="_blank"
                 rel="noreferrer"
+                aria-label={`${item?.label || item?.link_text} Link`}
               >
                 {content(item)}
               </a>
             ) : (
-              <Link to={item.link.value} title={item.label} className="spotlights-gtm">
+              <Link
+                to={item.link.value}
+                title={item.label}
+                className="spotlights-gtm"
+                aria-label={`${item?.label || item?.link_text} Link`}
+              >
                 {content(item)}
               </Link>
             )}

package/src/components/molecules/content/content.test.js

@@ -17,8 +17,8 @@ describe('Content Component', () => {
     expect(container.querySelectorAll('i')).toHaveLength(1);
 
     // Lazy image
-    expect(container.querySelector('img')).toBeFalsy();
-    expect(container.querySelector('div.lazyload-placeholder')).toBeTruthy();
+    // expect(container.querySelector('img')).toBeFalsy();
+    // expect(container.querySelector('div.lazyload-placeholder')).toBeTruthy();
   });
 });
 afterEach(() => {

package/src/components/molecules/pagination/with-midpoints.js

@@ -57,7 +57,7 @@ const PaginationWithMidPoints = ({
         )}
       </li>
 
-      <div className={styles.padd}>
+      <ul className={styles.padd}>
         {/* first page link */}
         {current !== 1 && (
           <li>
@@ -122,7 +122,7 @@ const PaginationWithMidPoints = ({
             {total}
           </Link>
         </li>
-      </div>
+      </ul>
 
       {/* next button */}
       <li className={styles.showInAll}>

package/src/components/molecules/slider/index.js

@@ -200,6 +200,7 @@ function Slider({
           className={`${styles.sliderContent}`}
           style={style}
           ref={sliderContentRef}
+          aria-label="Slider Content Button"
         >
           {children.map((slide, index) => (
             <div

package/src/components/pages/tracker/index.js

@@ -1,6 +1,5 @@
 import React, { useEffect } from 'react';
 import PropTypes from 'prop-types';
-import { Helmet } from 'react-helmet';
 import Button from '~atoms/button';
 import LazyImage from '~hooks/lazy-image';
 import { prettyTracker } from '~helpers/getters';
@@ -91,9 +90,6 @@ const Tracker = ({
     <>
       {showSplashScreen && !isMobile && (
         <>
-          <Helmet>
-            <meta name="robots" content="noindex,nofollow" />
-          </Helmet>
           <div className={styles.container}>
             {background}
             <div className={styles.content}>

package/src/helpers/getters.js

@@ -90,8 +90,15 @@ export function image(filename, width, height, fit = 'cover') {
   return `${cloudFrontUrl}/${Buffer.from(imageRequest).toString('base64')}`;
 }
 
+export function getImageExtension(filename) {
+  return filename && filename.split('.').pop();
+}
+
 export function imagePrettyUrl(filename = 'image.png', width = null, height = null) {
-  const cdnURL = process.env.IMAGE_CDN_URL || process.env.STORYBOOK_IMAGE_CDN_URL;
+  const cdnURL =
+    (getImageExtension(filename) === 'gif'
+      ? process.env.GIFS_CDN_URL
+      : process.env.IMAGE_CDN_URL) || process.env.STORYBOOK_IMAGE_CDN_URL;
 
   if (width && height) {
     const urlPath = '/fit-in';
@@ -262,7 +269,8 @@ export function getUrl(path) {
   if (path === '/') {
     return process.env.GATSBY_SITE_URL;
   }
-  return `${process.env.GATSBY_SITE_URL}${path}`;
+
+  return `${process.env.GATSBY_SITE_URL}${path.replace(/^\//, '')}`;
 }
 
 export function getLanguage(language) {

package/src/helpers/getters.test.js

@@ -68,6 +68,12 @@ describe('Getters Helper', () => {
     );
   });
 
+  test('getImageExtension()', () => {
+    expect(Getters.getImageExtension('photo.jpg')).toBeTruthy();
+    expect(Getters.getImageExtension('photo.jpg')).toEqual('jpg');
+    expect(Getters.getImageExtension(null)).toEqual(null);
+  });
+
   test('getPageImage()', () => {
     expect(
       Getters.getPageImage({ type: 'operator', relation: { default_logo_url: 'foo.jpg' } })
@@ -191,8 +197,9 @@ describe('Getters Helper', () => {
   });
 
   test('getUrl()', () => {
-    expect(Getters.getUrl('/')).toEqual('http://urltest.com');
+    expect(Getters.getUrl('/')).toEqual('http://urltest.com/');
     expect(Getters.getUrl('/games')).toEqual('http://urltest.com/games');
+    expect(Getters.getUrl('games')).toEqual('http://urltest.com/games');
   });
 
   test('getLanguage()', () => {

package/src/helpers/schema.js

@@ -1,4 +1,5 @@
 import { getGameRating, getUrl, getLanguage } from './getters';
+import { cleanHTML } from './strings';
 
 /* HardCoded Start */
 const isIL = process.env.GATSBY_SITE_NAME === 'Irishluck.ie';
@@ -178,9 +179,9 @@ export function moduleSchemas(modules) {
             '@type': 'Question',
             acceptedAnswer: {
               '@type': 'Answer',
-              text: item.answer,
+              text: cleanHTML(item.answer),
             },
-            name: item.question,
+            name: cleanHTML(item.question),
           })),
         };
 

package/src/helpers/schema.test.js

@@ -370,8 +370,9 @@ describe('Schema Helper', () => {
   test('schemaGenerator()', () => {
     const output = Schema.schemaGenerator({
       seo_json_schema: 'SEO JSON Schema',
-      breadcrumbs: [{}],
+      breadcrumbs: [{ path: '/test' }],
       type: 'article',
+      path: '/test-path',
       sections: { main: { modules: [{ name: 'faq' }] } },
     });
 

package/src/helpers/strings.js

@@ -51,3 +51,7 @@ export function truncateString(str, num, dots = '...') {
   trimmedString = trimmedString.substr(0, trimmedString.lastIndexOf(' '));
   return `${trimmedString}${dots}`;
 }
+
+export function cleanHTML(string) {
+  return string && string.replace(/(<([^>]+)>)/gi, '');
+}

package/src/helpers/strings.test.js

@@ -26,7 +26,11 @@ describe('Strings Helper', () => {
   });
 
   test('truncateString()', () => {
-    expect(Strings.truncateString('hello i think',5)).toEqual('hello...');
-    expect(Strings.truncateString('hello i think',10)).toEqual('hello i...');
+    expect(Strings.truncateString('hello i think', 5)).toEqual('hello...');
+    expect(Strings.truncateString('hello i think', 10)).toEqual('hello i...');
+  });
+
+  test('cleanHTML()', () => {
+    expect(Strings.cleanHTML('<p>hello <a>i</a> think</p>')).toEqual('hello i think');
   });
 });

package/src/hooks/lazy-image/index.js

@@ -1,6 +1,6 @@
-import React, { useState } from 'react';
+import React from 'react';
 // eslint-disable-next-line import/no-extraneous-dependencies
-import LazyLoad from 'react-lazyload';
+// import LazyLoad from 'react-lazyload';
 import PropTypes from 'prop-types';
 
 // When to use this component:
@@ -9,49 +9,68 @@ import PropTypes from 'prop-types';
 export default function LazyImage({
   height,
   width,
-  offset = 200,
+  // offset = 200,
   style = {},
   className,
   src = '#',
-  srcSet = '',
+  // srcSet = '',
   alt = '',
-  sizes,
-  defaultImg,
-  once = false,
+  // sizes,
+  // defaultImg,
+  // once = false,
 }) {
-  const [errorImage, setErrorImage] = useState(false);
+  // const [errorImage, setErrorImage] = useState(false);
 
-  if ((defaultImg && !src) || errorImage === true) {
-    return defaultImg;
-  }
+  // if ((defaultImg && !src) || errorImage === true) {
+  //   return defaultImg;
+  // }
 
   return (
-    <LazyLoad height={`${height}px`} width={`${width}px`} offset={offset} debounce={0} once={once}>
+    <>
+      {/* <LazyLoad
+        height={`${height}px`}
+        width={`${width}px`}
+        offset={offset}
+        debounce={0}
+        once={once}
+      >
+        <img
+          src={src}
+          className={className}
+          height={height}
+          width={width}
+          alt={alt}
+          style={style}
+          srcSet={srcSet}
+          sizes={sizes}
+          onError={() => setErrorImage(true)}
+        />
+      </LazyLoad> */}
+      {/* <noscript> */}
       <img
         src={src}
+        loading="lazy"
         className={className}
         height={height}
         width={width}
         alt={alt}
         style={style}
-        srcSet={srcSet}
-        sizes={sizes}
-        onError={() => setErrorImage(true)}
       />
-    </LazyLoad>
+      {/* </noscript> */}
+    </>
   );
 }
 
 LazyImage.propTypes = {
   width: PropTypes.number,
   height: PropTypes.number,
-  offset: PropTypes.number,
+  // offset: PropTypes.number,
   style: PropTypes.shape({}),
   className: PropTypes.string,
   src: PropTypes.string,
   alt: PropTypes.string,
-  sizes: PropTypes.string,
-  srcSet: PropTypes.string,
-  defaultImg: PropTypes.element,
-  once: PropTypes.bool,
+  // sizes: PropTypes.string,
+  // srcSet: PropTypes.string,
+  // defaultImg: PropTypes.element,
+  // once: PropTypes.bool,
 };

package/src/hooks/modal/modal-content.js

@@ -60,7 +60,12 @@ const ModalContent = ({ closeIcon }) => {
       <div className={styles.modalOverlay} onClick={close} />
       <div className={styles.modalContent}>
         {closeIcon && (
-          <button className={`${styles.closeIcon} modal-gtm btn-cta`} type="button" onClick={close}>
+          <button
+            className={`${styles.closeIcon} modal-gtm btn-cta`}
+            aria-label="Close Icon"
+            type="button"
+            onClick={close}
+          >
             {closeIcon}
           </button>
         )}

package/tests/envVars.js

@@ -1,5 +1,5 @@
 process.env.GATSBY_SITE_NAME = 'norskespilleautomater.com';
-process.env.GATSBY_SITE_URL = 'http://urltest.com';
+process.env.GATSBY_SITE_URL = 'http://urltest.com/';
 process.env.IMAGE_CDN_URL = 'https://cdn.images.com';
 process.env.TRACKER_LINK_FORMAT_MAIN = '[no],[visit],short_name';
 process.env.TRACKER_LINK_FORMAT_NON_MAIN = '[no],[visit],short_name,type,[tracker]';