From ca1a8e91326bbc59b63874417df41039b1e1dc90 Mon Sep 17 00:00:00 2001
From: crschnick <crschnick@xpipe.io>
Date: Fri, 13 Sep 2024 03:48:43 +0000
Subject: [PATCH] Make window close async

---
 .../java/io/xpipe/app/core/mode/OperationMode.java    | 11 +++++++++++
 .../java/io/xpipe/app/core/window/AppMainWindow.java  | 11 +++++------
 2 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/app/src/main/java/io/xpipe/app/core/mode/OperationMode.java b/app/src/main/java/io/xpipe/app/core/mode/OperationMode.java
index ac3a76040..fd5b6c206 100644
--- a/app/src/main/java/io/xpipe/app/core/mode/OperationMode.java
+++ b/app/src/main/java/io/xpipe/app/core/mode/OperationMode.java
@@ -254,6 +254,17 @@ public abstract class OperationMode {
         Runtime.getRuntime().halt(code);
     }
 
+    public static void onWindowClose() {
+        if (AppPrefs.get() == null) {
+            return;
+        }
+
+        var action = AppPrefs.get().closeBehaviour().getValue();
+        ThreadHelper.runAsync(() -> {
+            action.run();
+        });
+    }
+
     public static void shutdown(boolean inShutdownHook, boolean hasError) {
         // We can receive shutdown events while we are still starting up
         // In that case ignore them until we are finished
diff --git a/app/src/main/java/io/xpipe/app/core/window/AppMainWindow.java b/app/src/main/java/io/xpipe/app/core/window/AppMainWindow.java
index 6e78bf98e..025100903 100644
--- a/app/src/main/java/io/xpipe/app/core/window/AppMainWindow.java
+++ b/app/src/main/java/io/xpipe/app/core/window/AppMainWindow.java
@@ -4,6 +4,7 @@ import io.xpipe.app.core.AppCache;
 import io.xpipe.app.core.AppImages;
 import io.xpipe.app.core.AppProperties;
 import io.xpipe.app.core.AppTheme;
+import io.xpipe.app.core.mode.OperationMode;
 import io.xpipe.app.fxcomps.Comp;
 import io.xpipe.app.issue.ErrorEvent;
 import io.xpipe.app.issue.TrackEvent;
@@ -11,7 +12,6 @@ import io.xpipe.app.prefs.AppPrefs;
 import io.xpipe.app.prefs.CloseBehaviourAlert;
 import io.xpipe.app.util.ThreadHelper;
 import io.xpipe.core.process.OsType;
-
 import javafx.beans.property.BooleanProperty;
 import javafx.beans.property.SimpleBooleanProperty;
 import javafx.geometry.Rectangle2D;
@@ -24,18 +24,17 @@ import javafx.scene.layout.Region;
 import javafx.scene.paint.Color;
 import javafx.stage.Screen;
 import javafx.stage.Stage;
-
 import lombok.Builder;
 import lombok.Getter;
 import lombok.Value;
 import lombok.extern.jackson.Jacksonized;
 
+import javax.imageio.ImageIO;
 import java.io.IOException;
 import java.nio.file.Path;
 import java.time.Duration;
 import java.time.Instant;
 import java.time.temporal.ChronoUnit;
-import javax.imageio.ImageIO;
 
 public class AppMainWindow {
 
@@ -167,14 +166,14 @@ public class AppMainWindow {
             // Close other windows
             Stage.getWindows().stream().filter(w -> !w.equals(stage)).toList().forEach(w -> w.fireEvent(e));
             stage.close();
-            AppPrefs.get().closeBehaviour().getValue().run();
+            OperationMode.onWindowClose();
             e.consume();
         });
 
         stage.addEventHandler(KeyEvent.KEY_PRESSED, event -> {
             if (new KeyCodeCombination(KeyCode.Q, KeyCombination.SHORTCUT_DOWN).match(event)) {
                 stage.close();
-                AppPrefs.get().closeBehaviour().getValue().run();
+                OperationMode.onWindowClose();
                 event.consume();
             }
         });
@@ -277,7 +276,7 @@ public class AppMainWindow {
         if (OsType.getLocal().equals(OsType.LINUX) || OsType.getLocal().equals(OsType.MACOS)) {
             stage.getScene().addEventHandler(KeyEvent.KEY_PRESSED, event -> {
                 if (new KeyCodeCombination(KeyCode.W, KeyCombination.SHORTCUT_DOWN).match(event)) {
-                    AppPrefs.get().closeBehaviour().getValue().run();
+                    OperationMode.onWindowClose();
                     event.consume();
                 }
             });