fix(gui): second instance triggers second capture window #580

dynobo · Jan 14, 2024 · 9050099 · 9050099
1 parent e221fdc
commit 9050099
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/CHANGELOG b/CHANGELOG
@@ -2,9 +2,13 @@
 
 # Changelog
 
-## v0.5.4 (upcoming)
+## v0.5.4 (2024-01-15)
 
+- All: Fix duplicate capture window if started twice
+  ([#580](https://github.com/dynobo/normcap/issues/580))
 - Linux: Improve robustness of window positioning on Wayland.
+- Linux: Improve debug output regarding clipboard methods.
+  ([#581](https://github.com/dynobo/normcap/issues/581))
 
 ## v0.5.3 (2024-01-04)
 

diff --git a/normcap/gui/tray.py b/normcap/gui/tray.py
@@ -176,9 +176,14 @@ def _on_ready_read(self) -> None:
         """Process messages received from other NormCap instances."""
         if not self._socket_in:
             return
+
         message = self._socket_in.readAll()
+        # TODO: Test if `is message` check is necessary
         if message and message.data().decode("utf-8", errors="ignore") == "capture":
             logger.info("Received socket signal to capture.")
+            if self.windows:
+                logger.debug("Capture window(s) already open. Doing nothing.")
+                return
             self._show_windows(delay_screenshot=True)
 
     @QtCore.Slot(QtWidgets.QSystemTrayIcon.ActivationReason)