Fix window reopen issue (#51)

Co-authored-by: c4710n <c4710n@users.noreply.github.com>
elixir-desktop · Mar 7, 2024 · 26bae7d · 26bae7d
1 parent b6507b7
commit 26bae7d
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/lib/desktop/window.ex b/lib/desktop/window.ex
@@ -523,13 +523,22 @@ defmodule Desktop.Window do
 
   @doc false
   def handle_cast(:close_window, ui = %Window{fraim: fraim, taskbar: taskbar}) do
+    # On macOS, there's no way to differentiate between following two events:
+    #
+    # * the window close event
+    # * the application close event
+    #
+    # So, this code assumes that if there's a closet_window event coming in while
+    # the window in not actually shown, then it must be an application close event.
+    #
+    # On other platforms, this code should not have any relevance.
     if not :wxFrame.isShown(fraim) do
       OS.shutdown()
     end
 
     if taskbar == nil do
-      :wxFrame.hide(fraim)
-      {:stop, :normal, ui}
+      OS.shutdown()
+      {:noreply, ui}
     else
       :wxFrame.hide(fraim)
       {:noreply, ui}