From 3abf35ee58892bbc4306fe2529d31a9e3576d3bb Mon Sep 17 00:00:00 2001
From: Alexis <35.alexis.pele@gmail.com>
Date: Sat, 13 Jan 2024 14:53:37 +0100
Subject: [PATCH] Added functions to delete custom markers

---
 src/components/maps/WorldMap.astro            | 39 ++++++++++++++++++-
 src/components/maps/overlay/SearchMarkers.vue | 12 +++++-
 2 files changed, 49 insertions(+), 2 deletions(-)

diff --git a/src/components/maps/WorldMap.astro b/src/components/maps/WorldMap.astro
index ffb21c12..d359495b 100644
--- a/src/components/maps/WorldMap.astro
+++ b/src/components/maps/WorldMap.astro
@@ -296,6 +296,10 @@ map.setMaxBounds(
 // List to be persisted in localStorage
 let customMarkersList = []
 
+/**
+ * Function that saves the customMarkersList object into the localStorage
+ * It also sends an event to refresh data on other components that use localStorage
+ */
 function saveCustomMarkers() {
     localStorage.setItem('custom-markers', JSON.stringify(customMarkersList))
 
@@ -315,7 +319,7 @@ function storeLastHeldPositions(e) {
 
 /**
  * Function that creates custom markers and places them on the map
- * @param {string?} markerTitle
+ * @param {string} markerTitle
  */
 function addCustomMarker(markerTitle) {
     const lon = Number(localStorage.getItem('lastHeldXPosition'))
@@ -345,6 +349,7 @@ function addCustomMarker(markerTitle) {
 
     /**
      * Register event listener to fly to this custom marker
+     * TODO : Refactor these functions with the loadCustomMarkersFromStorage implementation
      */
     document.addEventListener(`fly-to-${markerTitle}`, (e) => {
         map.flyTo(customMarkerCoords, flyToZoomLevel, { duration: flyToDuration })
@@ -353,6 +358,14 @@ function addCustomMarker(markerTitle) {
         }, flyToDuration * 1000)
     })
 
+    /**
+     * Register event listener to remove this custom marker
+     * TODO : Refactor these functions with the loadCustomMarkersFromStorage implementation
+     */
+    document.addEventListener(`delete-${markerTitle}`, () => {
+        removeCustomMarker(customMarker, markerTitle)
+    })
+
     customMarkersList.push({
         lat,
         lon,
@@ -395,6 +408,7 @@ function loadCustomMarkersFromStorage() {
 
         /**
          * Register event listener to fly to this custom marker
+         * TODO : Refactor these functions with the addCustomMarker implementation
          */
         document.addEventListener(`fly-to-${m.title}`, (e) => {
             map.flyTo(savedMarkerCoords, flyToZoomLevel, { duration: flyToDuration })
@@ -402,10 +416,33 @@ function loadCustomMarkersFromStorage() {
                 savedMarker.openPopup()
             }, flyToDuration * 1000)
         })
+
+        /**
+         * Register event listener to remove this custom marker
+         * TODO : Refactor these functions with the addCustomMarker implementation
+         */
+        document.addEventListener(`delete-${m.title}`, () => {
+            removeCustomMarker(savedMarker, m.title)
+        })
     }
 }
 loadCustomMarkersFromStorage()
 
+/**
+ * Function that deletes a specific marker from storage and removes it from the map
+ * @param {Marker} marker Leaflet marker object
+ * @param {string} markerTitle Key for custom marker
+ */
+function removeCustomMarker(marker, markerTitle) {
+    marker.remove()
+
+    customMarkersList = customMarkersList.filter(m => {
+        return m.title !== markerTitle
+    })
+
+    saveCustomMarkers()
+}
+
 /**
  * UTILITIES
  */
diff --git a/src/components/maps/overlay/SearchMarkers.vue b/src/components/maps/overlay/SearchMarkers.vue
index c1e9db7e..ff05fbf4 100644
--- a/src/components/maps/overlay/SearchMarkers.vue
+++ b/src/components/maps/overlay/SearchMarkers.vue
@@ -18,6 +18,7 @@ const allMarkers = computed(() => {
 })
 
 // Search functions
+const searchBar = ref<HTMLDivElement>()
 const qInput = ref()
 const { focused: isFocused } = useFocus(qInput)
 
@@ -179,6 +180,15 @@ onMounted(() => {
         customMarkersData.value = useLocalStorage('custom-markers', []).value
     })
 })
+
+/**
+ * Emit event to delete custom marker
+ */
+function emitDeleteCustomMarker(markerTitle: string) {
+    const deleteCMarkerEvent = new CustomEvent(`delete-${markerTitle}`, { bubbles: true })
+
+    searchBar.value?.dispatchEvent(deleteCMarkerEvent)
+}
 </script>
 
 <template>
@@ -222,7 +232,7 @@ onMounted(() => {
                             <i class="ph-light ph-pencil-simple-line"></i>
                         </button> -->
 
-                        <button class="btn btn-danger btn-icon btn-sm">
+                        <button class="btn btn-danger btn-icon btn-sm" @click="emitDeleteCustomMarker(m.title)">
                             <i class="ph-light ph-trash"></i>
                         </button>
                     </div>