Pause mouse updates when resizing focused application

Author: huntfx

mousetracks2/components/processing.py

@@ -24,7 +24,7 @@
 from ..utils.monitor import MonitorData
 from ..utils.input import get_cursor_pos
 from ..utils.interface import Interfaces
-from ..utils.system import hide_child_process
+from ..utils.system import hide_child_process, UserResizeAppListener
 from ..constants import DEFAULT_PROFILE_NAME, UPDATES_PER_SECOND, DOUBLE_CLICK_MS, DOUBLE_CLICK_TOL, RADIAL_ARRAY_SIZE, DEBUG
 from ..render import render, EmptyRenderError, LayerBlend
 
@@ -69,6 +69,9 @@ def __post_init__(self) -> None:
         self._current_application = Application('', RectList())
         self.current_application = Application(DEFAULT_PROFILE_NAME, RectList())
 
+        self._resize_listener = UserResizeAppListener()
+        self._resize_listener.start()
+
     @property
     def timestamp(self) -> int:
         """Get the timestamp."""
@@ -166,6 +169,13 @@ def profile_age_days(self) -> int:
         current_day = self.timestamp // 86400
         return max(0, current_day - creation_day)
 
+    @property
+    def app_resizing(self) -> bool:
+        """Determine if the focused application is being resized."""
+        if self.current_application.name == DEFAULT_PROFILE_NAME:
+            return False
+        return self._resize_listener.triggered
+
     def _monitor_offset(self, pixel: tuple[int, int]) -> tuple[tuple[int, int], tuple[int, int]] | None:
         """Detect which monitor the pixel is on."""
         monitors = self.monitor_data.physical
@@ -617,14 +627,14 @@ def _process_message(self, message: ipc.Message) -> None:
                 print('[Processing] Render request completed')
 
             case ipc.MouseMove():
-                if not self.profile.config.track_mouse:
+                if not self.profile.config.track_mouse or self.app_resizing:
                     return
 
                 distance = self._record_move(self.profile.cursor_map, message.position)
                 self.profile.daily_distance[self.profile_age_days] += distance
 
             case ipc.MouseHeld():
-                if not self.profile.config.track_mouse:
+                if not self.profile.config.track_mouse or self.app_resizing:
                     return
 
                 result = self._monitor_offset(message.position)

mousetracks2/gui/main_window.py

@@ -38,6 +38,7 @@
 from ..utils.math import calculate_line, calculate_distance
 from ..utils.monitor import MonitorData
 from ..utils.system import SUPPORTS_TRAY, set_autostart, remove_autostart, split_autostart
+from ..utils.system import UserResizeAppListener
 from ..utils.update import is_latest_version, background_update
 
 if TYPE_CHECKING:
@@ -193,6 +194,7 @@ def __init__(self, component: GUI) -> None:
         self.component = component
         self.config = GlobalConfig()
 
+        # Set initial states
         self.pause_redraw = 0
         self.pause_colour_change = False
         self._pixel_redraw_queue: list[tuple[tuple[int, int] | None, tuple[int, int] | None, tuple[int, int] | None]] = []
@@ -220,6 +222,10 @@ def __init__(self, component: GUI) -> None:
         self._network_speed = NetworkSpeedStats()
         self.state = ipc.TrackingState.Paused
 
+        # Setup threads
+        self._resize_listener = UserResizeAppListener()
+        self._resize_listener.start()
+
         # Setup UI
         self.ui = layout.Ui_MainWindow()
         self.ui.setupUi(self)
@@ -853,7 +859,7 @@ def resolutions(self, resolutions: dict[tuple[int, int], tuple[int, bool]]) -> N
     def resolution_toggled(self, value: bool) -> None:
         """Toggle rendering of a particular resolution in a profile."""
         checkbox = cast(QtWidgets.QCheckBox, self.sender())
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return
         width, height = map(int, checkbox.text().split('x'))
@@ -875,7 +881,7 @@ def multi_monitor_change(self) -> None:
         """Change the multiple monitor option."""
         if not self.ui.opts_monitor.isChecked():
             return
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None:
             return
         self.component.send_data(ipc.ToggleProfileMultiMonitor(sanitised_profile_name, self.ui.multi_monitor.isChecked()))
@@ -885,7 +891,7 @@ def multi_monitor_override_toggle(self, checked: bool) -> None:
         """Enable or disable the multi monitor override."""
         if not self.ui.opts_monitor.isEnabled():
             return
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None:
             return
         self.component.send_data(ipc.ToggleProfileMultiMonitor(sanitised_profile_name, self.ui.multi_monitor.isChecked() if checked else None))
@@ -1030,7 +1036,7 @@ def manual_save_profile(self) -> None:
             return
         self.save_profile_request_sent = True
 
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is not None:
             self.component.send_data(ipc.Save(sanitised_profile_name))
 
@@ -1042,7 +1048,7 @@ def toggle_autosave(self, state: QtCore.Qt.CheckState) -> None:
     @QtCore.Slot(int)
     def profile_changed(self, idx: int) -> None:
         """Change the profile and trigger a redraw."""
-        sanitised_profile_name, profile_name = self._get_profile_data(idx)
+        sanitised_profile_name, profile_name = self._selected_profile_data(idx)
 
         if sanitised_profile_name is not None and not self._redrawing_profiles:
             self.request_profile_data(sanitised_profile_name)
@@ -1051,7 +1057,7 @@ def profile_changed(self, idx: int) -> None:
             self.set_profile_modified_text()
 
     def request_profile_data(self, sanitised_profile_name: str) -> None:
-        """Request loading profile data."""
+        """Request loading profile data from the background process."""
         self.component.send_data(ipc.ProfileDataRequest(sanitised_profile_name,
                                                         self._profile_names[sanitised_profile_name]))
 
@@ -1069,8 +1075,11 @@ def request_profile_data(self, sanitised_profile_name: str) -> None:
         self._is_loading_profile += 1
         self.start_rendering_timer()
 
-    def _get_profile_data(self, idx: int | None = None) -> tuple[str, str] | tuple[None, None]:
-        """Get the selected profile name from the combobox."""
+    def _selected_profile_data(self, idx: int | None = None) -> tuple[str, str] | tuple[None, None]:
+        """Get the selected profile name from the combobox.
+
+        Returns the sanitised profile name and actual name.
+        """
         if idx is None:
             sanitised_profile_name = self.ui.current_profile.currentData()
         else:
@@ -1309,7 +1318,7 @@ def _request_thumbnail(self) -> bool:
 
         width = self.ui.thumbnail.width()
         height = self.ui.thumbnail.height()
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None:
             return False
 
@@ -1346,7 +1355,7 @@ def _request_thumbnail(self) -> bool:
         return True
 
     def get_render_layer_data(self, file_path: str | None = None) -> Iterator[ipc.RenderLayer]:
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None:
             return
 
@@ -1473,7 +1482,7 @@ def request_render(self) -> None:
             case _:
                 name = 'Data'
 
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return
         profile_safe = re.sub(r'[^\w_.)( -]', '', profile_name)
@@ -1686,11 +1695,11 @@ def _process_message(self, message: ipc.Message) -> None:
                     im.save(message.request.file_path)
                     os.startfile(message.request.file_path)
 
-            case ipc.MouseHeld() if self.is_live and self.mouse_tracking_enabled:
+            case ipc.MouseHeld() if self.is_live and self.mouse_tracking_enabled and not self.app_resizing:
                 self.mouse_held_count += 1
 
             case ipc.MouseMove() if self.is_live and self.mouse_tracking_enabled:
-                if self.render_type == ipc.RenderType.MouseMovement:
+                if self.render_type == ipc.RenderType.MouseMovement and not self.app_resizing:
                     self.draw_pixmap_line(message.position, self.cursor_data.position)
                 self.update_track_data(self.cursor_data, message.position)
                 self.ui.stat_distance.setText(format_distance(self.cursor_data.distance))
@@ -2273,11 +2282,18 @@ def update_track_data(self, data: MapData, position: tuple[int, int]) -> None:
     @property
     def is_live(self) -> bool:
         """Determine if the visible data is live."""
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None:
             return False
         return sanitised_profile_name == sanitise_profile_name(self.current_profile.name)
 
+    @property
+    def app_resizing(self) -> bool:
+        """Determine if the focused application is being resized."""
+        if self.current_profile.name == DEFAULT_PROFILE_NAME:
+            return False
+        return self._resize_listener.triggered
+
     @property
     def mouse_tracking_enabled(self) -> bool:
         """Determine if mouse tracking is enabled."""
@@ -2361,7 +2377,7 @@ def update_thumbnail_size(self) -> None:
 
     def delete_mouse(self) -> None:
         """Request deletion of mouse data for the current profile."""
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return None
 
@@ -2385,7 +2401,7 @@ def delete_mouse(self) -> None:
 
     def delete_keyboard(self) -> None:
         """Request deletion of keyboard data for the current profile."""
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return None
 
@@ -2408,7 +2424,7 @@ def delete_keyboard(self) -> None:
 
     def delete_gamepad(self) -> None:
         """Request deletion of gamepad data for the current profile."""
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return None
 
@@ -2432,7 +2448,7 @@ def delete_gamepad(self) -> None:
 
     def delete_network(self) -> None:
         """Request deletion of network data for the current profile."""
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return None
 
@@ -2455,7 +2471,7 @@ def delete_network(self) -> None:
 
     def delete_profile(self) -> None:
         """Delete the selected profile."""
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return None
 
@@ -2484,7 +2500,7 @@ def delete_profile(self) -> None:
     def toggle_profile_mouse_tracking(self, state: QtCore.Qt.CheckState) -> None:
         if not self.ui.track_mouse.isEnabled():
             return
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None:
             return
 
@@ -2495,7 +2511,7 @@ def toggle_profile_mouse_tracking(self, state: QtCore.Qt.CheckState) -> None:
     def toggle_profile_keyboard_tracking(self, state: QtCore.Qt.CheckState) -> None:
         if not self.ui.track_keyboard.isEnabled():
             return
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None:
             return
 
@@ -2506,7 +2522,7 @@ def toggle_profile_keyboard_tracking(self, state: QtCore.Qt.CheckState) -> None:
     def toggle_profile_gamepad_tracking(self, state: QtCore.Qt.CheckState) -> None:
         if not self.ui.track_gamepad.isEnabled():
             return
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None:
             return
 
@@ -2517,7 +2533,7 @@ def toggle_profile_gamepad_tracking(self, state: QtCore.Qt.CheckState) -> None:
     def toggle_profile_network_tracking(self, state: QtCore.Qt.CheckState) -> None:
         if not self.ui.track_network.isEnabled():
             return
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None:
             return
 
@@ -2572,7 +2588,7 @@ def mark_profiles_unsaved(self, *profile_names: str) -> None:
 
     def set_profile_modified_text(self) -> None:
         """Set the text if the profile has been modified."""
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is not None and sanitised_profile_name in self._unsaved_profiles:
             self.ui.profile_modified.setText('Yes')
             self.ui.profile_save.setEnabled(self.state != ipc.TrackingState.Stopped)
@@ -2788,7 +2804,7 @@ def export_mouse_stats(self) -> None:
         dialog.setNameFilters(['CSV Files (*.csv)"'])
         dialog.setDefaultSuffix('csv')
 
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return
         profile_safe = re.sub(r'[^\w_.)( -]', '', profile_name)
@@ -2806,7 +2822,7 @@ def export_keyboard_stats(self) -> None:
         dialog.setNameFilters(['CSV Files (*.csv)"'])
         dialog.setDefaultSuffix('csv')
 
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return
         profile_safe = re.sub(r'[^\w_.)( -]', '', profile_name)
@@ -2824,7 +2840,7 @@ def export_gamepad_stats(self) -> None:
         dialog.setNameFilters(['CSV Files (*.csv)"'])
         dialog.setDefaultSuffix('csv')
 
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return
         profile_safe = re.sub(r'[^\w_.)( -]', '', profile_name)
@@ -2842,7 +2858,7 @@ def export_network_stats(self) -> None:
         dialog.setNameFilters(['CSV Files (*.csv)"'])
         dialog.setDefaultSuffix('csv')
 
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return
         profile_safe = re.sub(r'[^\w_.)( -]', '', profile_name)
@@ -2860,7 +2876,7 @@ def export_daily_stats(self) -> None:
         dialog.setNameFilters(['CSV Files (*.csv)"'])
         dialog.setDefaultSuffix('csv')
 
-        sanitised_profile_name, profile_name = self._get_profile_data()
+        sanitised_profile_name, profile_name = self._selected_profile_data()
         if sanitised_profile_name is None or profile_name is None:
             return
         profile_safe = re.sub(r'[^\w_.)( -]', '', profile_name)

mousetracks2/utils/system/__init__.py

@@ -11,6 +11,7 @@
     MonitorEventListener: Type[base.EventListener]
     ControllerEventListener: Type[base.EventListener]
     ForegroundAppListener: Type[base.EventListener]
+    UserResizeAppListener: Type[base.EventListener]
 
 match sys.platform:
     case 'win32':
@@ -19,7 +20,8 @@
         from .windows import get_autostart, set_autostart, remove_autostart
         from .windows import is_elevated, relaunch_as_elevated
         from .windows import Window
-        from .windows import MonitorEventListener, ControllerEventListener, ForegroundAppListener
+        from .windows import MonitorEventListener, ControllerEventListener
+        from .windows import ForegroundAppListener, UserResizeAppListener
         from .base import hide_child_process
         from .windows import prepare_application_icon
         from .windows import update_installer_version_number
@@ -30,7 +32,8 @@
         from .macos import get_autostart, set_autostart, remove_autostart
         from .base import is_elevated, relaunch_as_elevated
         from .macos import Window
-        from .base import MonitorEventListener, ControllerEventListener, ForegroundAppListener
+        from .base import MonitorEventListener, ControllerEventListener
+        from .base import ForegroundAppListener, UserResizeAppListener
         from .macos import hide_child_process, prepare_application_icon
         from .base import update_installer_version_number
 
@@ -40,7 +43,8 @@
         from .linux import get_autostart, set_autostart, remove_autostart
         from .base import is_elevated, relaunch_as_elevated
         from .linux import Window
-        from .base import MonitorEventListener, ControllerEventListener, ForegroundAppListener
+        from .base import MonitorEventListener, ControllerEventListener
+        from .base import ForegroundAppListener, UserResizeAppListener
         from .base import hide_child_process, prepare_application_icon
         from .base import update_installer_version_number
 
@@ -50,7 +54,8 @@
     'get_autostart', 'set_autostart', 'remove_autostart', 'remap_autostart',
     'is_elevated', 'relaunch_as_elevated',
     'Window',
-    'MonitorEventListener', 'ControllerEventListener', 'ForegroundAppListener',
+    'MonitorEventListener', 'ControllerEventListener',
+    'ForegroundAppListener', 'UserResizeAppListener',
     'hide_child_process', 'prepare_application_icon',
     'update_installer_version_number',
 ]