Move some tracking calls so they only happen on a user action. Couple other tracking updates.

Author: wdamien

dev/src/Flavor.js

@@ -37,7 +37,7 @@ export default class Flavor extends EventDispatcher {
 	set value(id) {
 		let profile = profiles[(id && id.toLowerCase()) || "js"];
 		if (!profile || profile === this._profile) { return; }
-		Track.page("flavor/"+id);
+
 		this._profile = profile;
 		this._buildSupportMap(profile);
 		app.prefs.write("flavor", id);

dev/src/views/Expression.js

@@ -64,7 +64,6 @@ export default class Expression extends EventDispatcher {
 
 	set flags(flags) {
 		flags = app.flavor.validateFlagsStr(flags);
-		Track.event("set_flags", "engagement", {flags:flags});
 		let str = this.editor.getValue(), index = str.lastIndexOf(this.delim);
 		this.editor.replaceRange(flags, {line: 0, ch: index + 1}, {line: 0, ch: str.length }); // this doesn't work if readOnly is false.
 	}
@@ -191,11 +190,13 @@ export default class Expression extends EventDispatcher {
 	_onFlavorListChange() {
 		app.tooltip.toggle.hide("flavor");
 		app.flavor.value = this.flavorList.selected;
+		Track.page("flavor/"+this.flavorList.selected);
 	}
 
 	_onFlagListChange() {
 		let sel = this.flagsList.selected;
 		this.flags = sel ? sel.join("") : "";
+		Track.event("set_flags", "engagement", this.flags);
 	}
 
 	_onFlavorChange() {

dev/src/views/Sidebar.js

@@ -66,7 +66,6 @@ export default class Sidebar {
 		if (!item || item.id === "menu") { return; } // expand button on the min menu
 		this.minimize(false);
 		if (item.id) {
-			Track.page("sidebar/"+item.id);
 			if (item.id === "home" || (item.parent && item.parent.id === "home") || this._isInReference(item)) {
 				app.prefs.write("side", item.id);
 			}
@@ -152,7 +151,13 @@ export default class Sidebar {
 		// list & content:
 		this.listEl = $.query("> .list", this.fullEl);
 		this.menuList = new List(this.listEl, {data:content.kids, template:this.menuListTemplate});
-		this.menuList.on("change", ()=> this.show(this.menuList.selectedItem));
+		this.menuList.on("change", () => {
+			const lastId = this.curItem.id;
+			this.show(this.menuList.selectedItem);
+			if (lastId !== this.curItem.id) {
+				Track.page("sidebar/"+this.curItem.id);
+			}
+		});
 		this.menuList.on("dblclick", ()=> this._onDblClick(this.menuList.selectedItem));
 		this.contentEl = $.query("> .content", this.fullEl);
 
@@ -162,8 +167,8 @@ export default class Sidebar {
 
 		let template = $.template`<svg class="icon"><use xlink:href="#${"id"}"></use></svg>`;
 		this.minList = new List($.query("> .list", this.minEl), {template});
-		this.minList.on("change", (evt)=> { this.show(this.minList.selectedItem); evt.preventDefault(); });
-		
+		this.minList.on("change", (evt)=> { this.show(this.minList.selectedItem); evt.preventDefault(); Track.page("sidebar/"+this.curItem.id); });
+
 		// set up special content:
 		this.community = new Community($.query("#library > #community"));
 		this.share = new Share($.query("#library > #share"));

dev/src/views/Text.js

@@ -32,6 +32,7 @@ import TextHighlighter from "./TextHighlighter";
 import TextHover from "./TextHover";
 import EventDispatcher from "../events/EventDispatcher";
 import List from "../controls/List";
+import Track from "../utils/Track";
 
 import app from "../app";
 
@@ -155,6 +156,8 @@ export default class Text extends EventDispatcher {
 		this.dispatchEvent("modechange");
 		if (this.mode === "text") { this.editor.refresh(); }
 		else { $.addClass(this.el, "tests-viewed"); }
+
+		Track.page("mode/"+this.mode);
 	}
 
 	_setResult(val) {

dev/src/views/Tools.js

@@ -69,7 +69,6 @@ export default class Tools extends EventDispatcher {
 
 	show(id) {
 		if (!id || id === this._toolId) { return; }
-		Track.page("tool/"+id);
 
 		this.toolList.selected = this._toolId = id;
 		let input = (id === "replace" || id === "list");
@@ -138,6 +137,7 @@ export default class Tools extends EventDispatcher {
 
 	_handleListChange() {
 		this.show(this.toolList.selected);
+		Track.page("tool/"+this.toolList.selected);
 	}
 
 	_handleHeaderClick(evt) {

index.html

@@ -5,7 +5,7 @@
 	<title>RegExr: Learn, Build, &amp; Test RegEx</title>
 
 	<!-- Global Site Tag (gtag.js) - Google Analytics -->
-	<script async src="https://www.googletagmanager.com/gtag/js?id=GA_TRACKING_ID"></script>
+	<script async src="https://www.googletagmanager.com/gtag/js?id=UA-3579542-6"></script>
 	<script>
 	window.dataLayer = window.dataLayer || [];
 	function gtag(){dataLayer.push(arguments)};