From bc863114b705afc4f75ecc6a4950cb646d734706 Mon Sep 17 00:00:00 2001
From: mattk70 <email@mattkirkland.co.uk>
Date: Sat, 16 Mar 2024 14:33:47 +0000
Subject: [PATCH] Updated map click handler with delay. Cancels click handler
 on doubleclick.

Track keyboard shortcuts
---
 index.html | 12 ++++++++++--
 js/ui.js   |  2 ++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/index.html b/index.html
index 5ceb2631..16b474d4 100644
--- a/index.html
+++ b/index.html
@@ -1187,7 +1187,7 @@ <h5 class="modal-title" id="locationModalLabel">Set Location</h5>
 <script src="./js/ui.js" type="module"></script>
 <script type="module" defer>
     import { config, displayLocationAddress, LOCATIONS } from './js/ui.js';
-    let map, marker;
+    let map, marker, clickTimer = 0;
     async function onMapClick(e) {
         const {lat, lng} = e.latlng;
         const target = map.getContainer().id;
@@ -1278,7 +1278,15 @@ <h5 class="modal-title" id="locationModalLabel">Set Location</h5>
         const latEl = document.getElementById(latTxt);
         const lonEl = document.getElementById(lonTxt);
         showMap(divID);
-        map.on('click', onMapClick);
+        map.on('click', (e) => {
+            clearTimeout(clickTimer);
+            //Wait for doubleclick
+            clickTimer = setTimeout(() => {onMapClick(e)}, 250);
+        })
+        // don't call onMapClick if doubleclicked
+        map.on('dblclick', (e) => {
+            clearTimeout(clickTimer);
+        })
     }
 
     window.placeMap = placeMap;
diff --git a/js/ui.js b/js/ui.js
index c29cf784..3e6de4b8 100644
--- a/js/ui.js
+++ b/js/ui.js
@@ -2181,6 +2181,8 @@ function onChartData(args) {
         let action = e.code;
         if (action in GLOBAL_ACTIONS) {
             if (document === e.target || document.body === e.target || e.target.attributes["data-action"]) {}
+            const modifier = e.shiftKey ? 'Shift' : e.ctrlKey ? 'Control' : e.metaKey ? 'Alt' : 'no';
+            trackEvent(config.UUID, 'KeyPress', action, modifier );
             GLOBAL_ACTIONS[action](e);
         }