From 7d80678b38362fb1d83065ba962579a7a8379f40 Mon Sep 17 00:00:00 2001
From: webadderall <131426131+webadderall@users.noreply.github.com>
Date: Thu, 9 Apr 2026 21:23:48 +1000
Subject: [PATCH] fix(windows): use per-display scale factor for window cursor
 position

When recording a window on a secondary monitor with a different DPI,
getNormalizedCursorPoint() used the primary display's scale factor for
both cursor and window-bounds conversion. This produced incorrect
normalised coordinates. Now resolves the display that contains the
target window and uses that display's scale factor instead.

Fixes #204
---
 electron/ipc/handlers.ts | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/electron/ipc/handlers.ts b/electron/ipc/handlers.ts
index d9284a86..4a08447b 100644
--- a/electron/ipc/handlers.ts
+++ b/electron/ipc/handlers.ts
@@ -3438,16 +3438,26 @@ function getNormalizedCursorPoint() {
   // physical pixel coordinates, while Electron's getCursorScreenPoint() and
   // display.bounds return DIP (logical) coordinates. Apply a DPI correction
   // so all values are in the same coordinate space before normalizing.
-  const sf = process.platform !== 'darwin'
+  // Use the display containing the window (or cursor) rather than the primary
+  // display so multi-monitor setups with different DPI scales work correctly.
+  const primarySf = process.platform !== 'darwin'
     ? (getScreen().getPrimaryDisplay().scaleFactor || 1)
     : 1
 
   const cursor = isLinuxCacheFresh
-    ? { x: linuxCursorCache.x / sf, y: linuxCursorCache.y / sf }
+    ? { x: linuxCursorCache.x / primarySf, y: linuxCursorCache.y / primarySf }
     : fallbackCursor
 
   const windowBounds = selectedSource?.id?.startsWith('window:') ? selectedWindowBounds : null
   if (windowBounds) {
+    // Resolve the scale factor for the display that contains the target window
+    // centre point, falling back to the primary display scale factor.
+    const sf = process.platform !== 'darwin'
+      ? (getScreen().getDisplayNearestPoint({
+          x: windowBounds.x / primarySf,
+          y: windowBounds.y / primarySf,
+        }).scaleFactor || 1)
+      : 1
     const width = Math.max(1, windowBounds.width / sf)
     const height = Math.max(1, windowBounds.height / sf)