Move google tagmanager noscript in body

1. noscript elements are not supported in the head. According to GTM docs, this should be placed directly below the <body> tag. This isn't exactly under the <body> tag, but it is high enough in the vulcan dom that it will work the same.
iFixit · Jul 5, 2023 · 26f68ea · 26f68ea
1 parent ddcb10f
commit 26f68ea
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/frontend/templates/troubleshooting/components/TagManager.tsx b/frontend/templates/troubleshooting/components/TagManager.tsx
@@ -21,7 +21,7 @@ function GoogleScript() {
    return <script dangerouslySetInnerHTML={{ __html: scriptTag }} />;
 }
 
-function GoogleNoScript() {
+export function GoogleNoScript() {
    return (
       <noscript>
          <iframe
@@ -74,7 +74,6 @@ export function TagManager() {
    return (
       <>
          <GoogleScript />
-         <GoogleNoScript />
          <MatomoScript />
       </>
    );

diff --git a/frontend/templates/troubleshooting/index.tsx b/frontend/templates/troubleshooting/index.tsx
@@ -52,7 +52,7 @@ import { BreadCrumbs } from '@ifixit/breadcrumbs';
 import { HeadingSelfLink } from './components/HeadingSelfLink';
 import ProblemCard from './Problem';
 import { PixelPing } from '@components/analytics/PixelPing';
-import { TagManager } from './components/TagManager';
+import { TagManager, GoogleNoScript } from './components/TagManager';
 
 const Wiki: NextPageWithLayout<{
    wikiData: TroubleshootingData;
@@ -87,6 +87,7 @@ const Wiki: NextPageWithLayout<{
 
    return (
       <Flex direction="column" alignItems="center" width="100%" fontSize="16px">
+         <GoogleNoScript />
          <NavBar
             editUrl={wikiData.editUrl}
             historyUrl={wikiData.historyUrl}