Debounce all tracking events

Author: Sambego

src/editor/index.js

@@ -41,7 +41,6 @@ import {
 } from "../dom-elements.js";
 
 import log from "loglevel";
-import _ from "lodash";
 
 // The event manager lets us enable/disable events as needed without
 // manually tracking them. Events that need to be disabled should be
@@ -439,18 +438,18 @@ function setupEvents() {
   eventManager.addCodeMirrorEvent(
     tokenEditor,
     "change",
-    _.throttle(decodeToken, 1000)
+    decodeToken
   );
   // When parts of the decoded token are changed, it must be reencoded.
   eventManager.addCodeMirrorEvent(
     headerEditor,
     "change",
-    _.throttle(encodeToken, 1000)
+    encodeToken
   );
   eventManager.addCodeMirrorEvent(
     payloadEditor,
     "change",
-    _.throttle(encodeToken, 1000)
+    encodeToken
   );
 
   // HMAC secret, show tooltip if secret is too short.

src/metrics.js

@@ -1,4 +1,6 @@
 import log from "loglevel";
+import _ from "lodash";
+
 
 export function init() {
   // Create a queue, but don't obliterate an existing one!
@@ -119,12 +121,12 @@ export function init() {
   analytics.page();
 }
 
-export function track(event, data) {
+export const track = _.debounce((event, data) => {
   if (window.metrics) {
     try {
       window.metrics.track(event, data);
     } catch (e) {
       log.error(`Metrics library error for event ${event}: ${e}`);
     }
   }
-}
+}, 2000);