misc(treemap): add GA snippet for new property #12481
Conversation
| @@ -87,17 +87,14 @@ | |||
|
|
|||
| <script src="src/bundled.js"></script> | |||
|
|
|||
| <!-- Google Analytics --> | |||
| <!-- Global site tag (gtag.js) - Google Analytics --> | |||
| <script async src="https://www.googletagmanager.com/gtag/js?id=G-16X003MB7V"></script> | |||
|
|
||
| ga('create', 'UA-85519014-2', 'auto'); |
There was a problem hiding this comment.
can we just change the property descriptor? analytics still documents this approach and I'd hate to not follow our own aspirational advice
There was a problem hiding this comment.
I don't know, and the different prefix in the identifier makes me think twice about doing that.
There was a problem hiding this comment.
well this majorly sucks https://support.google.com/analytics/answer/9539598?hl=en 😭
There was a problem hiding this comment.
OK, so GA 4 isn't as bad as I thought
good news: stopped doing the double script hop, so gtag is the only script loaded to fire GA events
bad news: the script is 140% larger and increases TBT 385% more (granted GA only contributes ~5ms with the original so quintupling only gives you ~25ms, but hey I'm still a little salty 😉 )
|
|
||
| ga('create', 'UA-85519014-2', 'auto'); |
There was a problem hiding this comment.
well this majorly sucks https://support.google.com/analytics/answer/9539598?hl=en 😭
Just tracking page views for now.
ref #11254