Skip to content

🐞 Use screen.displayBounds instead of screen.frame#850

Merged
mrkai77 merged 2 commits into
developfrom
835-window-jump-when-dragging-on-the-secondary-external-display
Oct 19, 2025
Merged

🐞 Use screen.displayBounds instead of screen.frame#850
mrkai77 merged 2 commits into
developfrom
835-window-jump-when-dragging-on-the-secondary-external-display

Conversation

@mrkai77

@mrkai77 mrkai77 commented Oct 18, 2025

Copy link
Copy Markdown
Owner

Loop was previously using the wrong display frame property to determine where to place a restored frame for a window. While the previous method worked mostly fine on single-display setups, this method did not work well on multi-display setups. This is because NSScreen.frame (which we were previously using) provides the screen frame in AppKit coordinate space, whose origin is not the same as what the accessibility API uses (CoreGraphics coordinate space). Using NSScreen.displayBounds will fix this.

@mrkai77 mrkai77 self-assigned this Oct 18, 2025
@mrkai77 mrkai77 linked an issue Oct 18, 2025 that may be closed by this pull request
4 tasks
@mrkai77 mrkai77 force-pushed the 835-window-jump-when-dragging-on-the-secondary-external-display branch from 61144f7 to a4056b4 Compare October 18, 2025 22:30
@mrkai77 mrkai77 added the Bug label Oct 18, 2025
@mrkai77 mrkai77 merged commit 764dd2e into develop Oct 19, 2025
1 check passed
@mrkai77 mrkai77 deleted the 835-window-jump-when-dragging-on-the-secondary-external-display branch October 19, 2025 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐞 window jump when dragging on the secondary (external) display

1 participant