From a397e7305d1572baf67823413f97a29eea38f0cc Mon Sep 17 00:00:00 2001
From: Ryan <43447928+Rmiller5466@users.noreply.github.com>
Date: Fri, 17 Oct 2025 10:45:36 -0400
Subject: [PATCH] fix: editor discard prompt doesn't save nor discard

Co-authored-by: Ryan Miller <ryan.miller@infinitetactics.com>
---
 .../components/prompts/DiscardEditorChanges.vue  | 10 ++--------
 frontend/src/views/files/Editor.vue              | 16 ++++++++++++++--
 2 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/frontend/src/components/prompts/DiscardEditorChanges.vue b/frontend/src/components/prompts/DiscardEditorChanges.vue
index bd6bc49a..7a99c5fc 100644
--- a/frontend/src/components/prompts/DiscardEditorChanges.vue
+++ b/frontend/src/components/prompts/DiscardEditorChanges.vue
@@ -17,7 +17,7 @@
       </button>
       <button
         class="button button--flat button--blue"
-        @click="saveAndClose"
+        @click="currentPrompt.saveAction"
         :aria-label="$t('buttons.saveChanges')"
         :title="$t('buttons.saveChanges')"
         tabindex="1"
@@ -39,8 +39,8 @@
 </template>
 
 <script>
-import { mapState, mapActions } from "pinia";
 import { useLayoutStore } from "@/stores/layout";
+import { mapActions, mapState } from "pinia";
 
 export default {
   name: "discardEditorChanges",
@@ -49,12 +49,6 @@ export default {
   },
   methods: {
     ...mapActions(useLayoutStore, ["closeHovers"]),
-    saveAndClose() {
-      if (this.currentPrompt?.saveAction) {
-        this.currentPrompt.saveAction();
-      }
-      this.closeHovers();
-    },
   },
 };
 </script>
diff --git a/frontend/src/views/files/Editor.vue b/frontend/src/views/files/Editor.vue
index f3d9aa94..0913ea98 100644
--- a/frontend/src/views/files/Editor.vue
+++ b/frontend/src/views/files/Editor.vue
@@ -216,12 +216,24 @@ const decreaseFontSize = () => {
 
 const close = () => {
   if (!editor.value?.session.getUndoManager().isClean()) {
-    layoutStore.showHover("discardEditorChanges");
+    layoutStore.showHover({
+      prompt: "discardEditorChanges",
+      confirm: (event: Event) => {
+        event.preventDefault();
+        finishClose();
+      },
+      saveAction: async () => {
+        await save();
+        finishClose();
+      },
+    });
     return;
   }
+  finishClose();
+};
 
+const finishClose = () => {
   fileStore.updateRequest(null);
-
   const uri = url.removeLastDir(route.path) + "/";
   router.push({ path: uri });
 };