Fix issue where rebounding occasionally picks the wrong target monitor

Crozzers · Jul 1, 2024 · c473d9f · c473d9f
1 parent 90b5dbd
commit c473d9f
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/common.py b/src/common.py
@@ -456,7 +456,9 @@ def set_pos(self, rect: Rect, placement: Optional[Placement] = None):
             offset = dpi_scale(self.get_border_and_shadow_thickness(), target_display_dpi)
 
             # check if Window will fit on the Display it's being moved to. If not, adjust the rect to fit
-            target_display_rect = self.get_closest_display_rect(rect[:2])
+            # use center point because top left might be out of bounds due to drop shadow and offset, which may
+            # lead to `MONITOR_DEFAULTTONEAREST` picking the wrong display
+            target_display_rect = self.get_closest_display_rect((rect[2] - rect[0], rect[3] - rect[1]))
             rect = self.rebound(rect, to_rect=target_display_rect, offset=offset)
 
             resizable = self.is_resizable()