Merge pull request #1000 from dbauszus-glx:confirm-changes

Confirm changes on close and edit toggle
GEOLYTIX · Nov 15, 2023 · 6bc6766 · 6bc6766
2 parents 9fd1106 + 3e499d5
commit 6bc6766
Showing 1 changed file with 33 additions and 10 deletions.
diff --git a/lib/ui/locations/view.mjs b/lib/ui/locations/view.mjs
@@ -99,16 +99,26 @@ export default location => {
       <button
         title = "Enable edits"
         class = ${`mask-icon edit ${location.new && 'on' || ''}`}
-        onclick = ${e => {
+        onclick = ${async e => {
 
         // Edits are on
         if (e.target.classList.contains('on')) {
 
           // Remove on class from button.
           e.target.classList.remove('on')
 
-          // Remove edits from infoj entries.
-          location.removeEdits()
+          if (location.infoj.some(entry => typeof entry.newValue !== 'undefined')) {
+
+            if (confirm(`Save your changes to this location?`)) {
+
+              await location.update()
+
+            } else {
+
+              // Remove edits from infoj entries.
+              location.removeEdits()
+            }
+          }
 
         } else {
 
@@ -170,9 +180,22 @@ export default location => {
     <button
       title = ${mapp.dictionary.location_remove}
       class = "mask-icon close no"
-      onclick = ${e => {
-      location.remove()
-      // location.layer.mapview.Map.updateSize()
+      onclick = ${async e => {
+
+        if (location.infoj.some(entry => typeof entry.newValue !== 'undefined')) {
+
+          if (confirm(`Save your changes to this location?`)) {
+
+            await location.update()
+
+          } else {
+
+            // The location will not be closed.
+            return;
+          }
+        }
+
+        location.remove()
     }}>`)
 
   location.view = mapp.ui.elements.drawer({
@@ -207,10 +230,10 @@ export default location => {
     }
 
     // Hide upload button if no other field in the infoj has a newValue.
-    location.view.querySelector('.btn-save')
-      .style.display = location.infoj
-        .some(entry => typeof entry.newValue !== 'undefined')
-      && 'inline-block' || 'none';
+    location.view.querySelector('.btn-save').style.display =
+      location.infoj.some(entry => typeof entry.newValue !== 'undefined')
+        ? 'inline-block'
+        : 'none';
 
   })