From 713189b4dce68d4b0162295d66eda2402cadd04d Mon Sep 17 00:00:00 2001 From: Yong Bakos Date: Wed, 3 Apr 2024 11:58:09 -0700 Subject: [PATCH] Layout: Adjust placement of ganalytics js The instructions from google are not clear and error-prone. Place the js within the head, at the end, not after the head. References #355 --- app/views/layouts/application.html.haml | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/app/views/layouts/application.html.haml b/app/views/layouts/application.html.haml index cac9cdd..ee26af3 100644 --- a/app/views/layouts/application.html.haml +++ b/app/views/layouts/application.html.haml @@ -17,13 +17,12 @@ = stylesheet_link_tag 'application', media: 'all', 'data-turbolinks-track': 'reload' = javascript_include_tag 'application', 'data-turbolinks-track': 'reload' = javascript_importmap_tags - / Google tag (gtag.js) should be after head. - %script{ src: "https://www.googletagmanager.com/gtag/js?id=G-PTHXK9D0S0"} - :javascript - window.dataLayer = window.dataLayer || []; - function gtag(){dataLayer.push(arguments);} - gtag('js', new Date()); - gtag('config', 'G-PTHXK9D0S0'); + %script{ src: "https://www.googletagmanager.com/gtag/js?id=G-PTHXK9D0S0"} + :javascript + window.dataLayer = window.dataLayer || []; + function gtag(){dataLayer.push(arguments);} + gtag('js', new Date()); + gtag('config', 'G-PTHXK9D0S0'); %body{class: "#{controller.action_name}"} = include_gon(:init => true) = render 'layouts/nav'