Refactor to use dialog close event for cleanup and promise resolution

Co-authored-by: haslinghuis <[email protected]>

Author: Copilot

src/js/gui.js

@@ -383,21 +383,26 @@ class GuiControl {
 
             const confirmAction = () => {
                 dialog[0].close();
-                $(document).off("keydown.informationDialog");
-                resolve();
             };
 
             buttonConfirm.on("click", confirmAction);
 
             // Add Enter key support for single-choice dialog
-            $(document).off("keydown.informationDialog");
-            $(document).on("keydown.informationDialog", (e) => {
+            const handleKeydown = (e) => {
                 // Only trigger if Enter is pressed, dialog is open, and target is not an input/textarea
                 if (e.which === 13 && dialog[0].open && !$(e.target).is("input, textarea")) {
                     e.preventDefault();
                     confirmAction();
                 }
-            });
+            };
+
+            $(document).on("keydown.informationDialog", handleKeydown);
+
+            // Clean up event listener when dialog closes
+            dialog[0].addEventListener("close", () => {
+                $(document).off("keydown.informationDialog", handleKeydown);
+                resolve();
+            }, { once: true });
 
             dialog[0].showModal();
         });