From 4f63a87a76fc070cdde3cd196dffefd1b20cd696 Mon Sep 17 00:00:00 2001
From: Austen Adler <agadler@austenadler.com>
Date: Sun, 19 Mar 2023 13:52:20 -0400
Subject: [PATCH] Add location control to the map properly

---
 web-frontend/src/lib/Map.svelte | 54 +++++++++++++++++++++------------
 web-frontend/src/lib/common.js  |  2 +-
 2 files changed, 35 insertions(+), 21 deletions(-)

diff --git a/web-frontend/src/lib/Map.svelte b/web-frontend/src/lib/Map.svelte
index 17da6a3..35a94a7 100644
--- a/web-frontend/src/lib/Map.svelte
+++ b/web-frontend/src/lib/Map.svelte
@@ -7,11 +7,17 @@
 
 	let mapElement;
 	let locationChecked;
+	let locationControl;
 
 	export let onMapClick = () => {};
 	export let map;
 
-	const locationCheckedChanged = () => {
+	const locationCheckedChanged = (setValue) => {
+		if (setValue === true || setValue === false) {
+			// Shortcut to set it in this function, if they passed a boolean
+			locationChecked = !!setValue;
+		}
+
 		if (locationChecked) {
 			console.log('Starting location');
 			map.locate({ setView: true, watch: true, enableHighAccuracy: true });
@@ -29,36 +35,40 @@
 			// TODO: Pick a better spot
 			map.setView([51.505, -0.09], 13);
 			map.on('locationerror', (e) => {
+				// If getting location caused an error, disable tracking
 				console.error('Error getting location', e);
-				locationChecked = false;
-				locationCheckedChanged();
+				locationCheckedChanged(false);
+			});
+			map.on('dragstart', (e) => {
+				// If they move the map at all, they are not interested in their location anymore
+				locationCheckedChanged(false);
+			});
+			map.on('click', (e) => {
+				// If they click the map at all, they are not interested in their location anymore
+				locationCheckedChanged(false);
 			});
 			map.on('click', onMapClick);
 			map.on('locationfound', (e) => {
 				dispatch('locationfound', e);
 			});
 
+			// Add the location control
+			let ctrl = leaflet.control();
+			ctrl.onAdd = () => {
+				return locationControl;
+			};
+			ctrl.update = (props) => {
+				console.info('Clicked ctrl:', props);
+			};
+
+			ctrl.addTo(map);
+
 			leaflet
 				.tileLayer('https://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png', {
 					attribution:
 						'&copy; <a href="https://www.openstreetmap.org/copyright">OpenStreetMap</a> contributors'
 				})
 				.addTo(map);
-
-			// let popup = leaflet.popup();
-
-			// map.on('click', (e) => {
-			// 	try {
-			// 		// TODO: Leaflet allows sending coordinates out of the standard range
-			// 		latlng = e.latlng;
-			// 		addr = address_from_lat_lon(latlng.lat, latlng.lng);
-			// 		popup.setLatLng(e.latlng).setContent(`${addr}`).openOn(map);
-			// 	} catch (err) {
-			// 		console.error(err);
-			// 		addr = '';
-			// 		popup.setLatLng(e.latlng).setContent(`You clicked at ${e.latlng}`).openOn(map);
-			// 	}
-			// });
 		}
 	});
 
@@ -71,9 +81,13 @@
 </script>
 
 <div id="container" class="h-3/4">
-	<label>
+	<label
+		bind:this={locationControl}
+		class="bg-white p-2 border-black border-solid"
+		on:click|stopPropagation
+	>
 		<input type="checkbox" bind:checked={locationChecked} on:change={locationCheckedChanged} />
-		Watch my location
+		Location
 	</label>
 	<div id="map" bind:this={mapElement} class="h-full w-full" />
 </div>
diff --git a/web-frontend/src/lib/common.js b/web-frontend/src/lib/common.js
index 84f75cb..6c4e619 100644
--- a/web-frontend/src/lib/common.js
+++ b/web-frontend/src/lib/common.js
@@ -4,7 +4,7 @@ import * as xpinWasm from 'xpin-wasm';
 export const emptyxpin = {
 	address: '',
 	latLon: [0.0, 0.0],
-	decimalDegrees: '',
+	decimalDegrees: ''
 };
 
 export function getxpin(xpin) {