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

Move Google Analytics import into the <head> section (#138)

Browse source 
* Move Google Analytics import into the <head> section

* Changes to move Google Tag Manager into <head> tag, while leaving the noscript section in the <body> tag.
This commit is contained in:
Eric Light 2017-09-21 08:34:36 +12:00 committed by Alexandre Vicenzi
parent 0091474a91
commit 229ad20d36
3 changed files with 13 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
templates/base.html
View file

@ -36,6 +36,10 @@
<link rel="icon" href="{{ FAVICON }}" type="image/x-icon">
{% endif %}
{% if GOOGLE_ANALYTICS %}
{% include "partial/ga.html" %}
{% endif %}
<meta charset="utf-8" />
<meta http-equiv="X-UA-Compatible" content="IE=edge" />
<meta name="HandheldFriendly" content="True" />
@ -75,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>
@ -169,10 +176,6 @@
</footer>
</main>
{% if GOOGLE_ANALYTICS %}
{% include "partial/ga.html" %}
{% endif %}
{% if GUAGES %}
{% include "partial/guages.html" %}
{% endif %}

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 -->