schtobia/flex
1
0
Fork 0
Code Issues Pull requests Activity

Changes to move Google Tag Manager into <head> tag, while leaving the noscript section in the <body> tag.

Browse source
This commit is contained in:
Eric Light 2017-09-20 21:09:42 +12:00
parent de64f4bd8b
commit c10e10e047
3 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
templates/base.html
View file

@ -79,10 +79,13 @@
});
</script>
{% endif %}
{% if GOOGLE_TAG_MANAGER %}
{% include "partial/gtm.html" %}
{% endif %}
</head>
<body>
{% if GOOGLE_TAG_MANAGER %}
{% include "partial/gtm.html" %}
{% include "partial/gtm_noscript.html" %}
{% endif %}
<aside>
<div>

3
templates/partial/gtm.html
View file

@ -1,7 +1,4 @@
<!-- Google Tag Manager -->
<noscript>
<iframe src="//www.googletagmanager.com/ns.html?id={{ GOOGLE_TAG_MANAGER }}" height="0" width="0" style="display:none;visibility:hidden"></iframe>
</noscript>
<script>
(function(w,d,s,l,i){
w[l]=w[l]||[];

5
templates/partial/gtm_noscript.html Normal file
View file

@ -0,0 +1,5 @@
<!-- Google Tag Manager -->
<noscript>
<iframe src="//www.googletagmanager.com/ns.html?id={{ GOOGLE_TAG_MANAGER }}" height="0" width="0" style="display:none;visibility:hidden"></iframe>
</noscript>
<!-- End Google Tag Manager -->