Skip to content

[06x] macos: fix multi-monitor relative pointer bug when first display is positioned to the right#4198

Merged
gonX merged 1 commit into
OpenTabletDriver:0.6.xfrom
AkiSakurai:relative_multi
Oct 5, 2025
Merged

[06x] macos: fix multi-monitor relative pointer bug when first display is positioned to the right#4198
gonX merged 1 commit into
OpenTabletDriver:0.6.xfrom
AkiSakurai:relative_multi

Conversation

@AkiSakurai
Copy link
Copy Markdown
Contributor

This commit removes the extra offset that was causing the cursor to jump away.
I'm not sure what the offset was for, perhaps it was added accidentally during the refactoring?

@github-actions github-actions Bot added macos Affects the MacOS platform desktop OpenTabletDriver.Desktop library, UX and Daemon use this as the core implementation. labels Oct 5, 2025
…y is positioned to the right

This commit removes the extra offset that was causing the cursor to jump away.
@gonX
Copy link
Copy Markdown
Member

gonX commented Oct 5, 2025

The change was added in #2764 and seems to go even further back. Does that help?

@gonX gonX self-requested a review October 5, 2025 01:54
@gonX gonX added the bug Something isn't working label Oct 5, 2025
@gonX gonX added this to the v0.6.6.1 milestone Oct 5, 2025
@AkiSakurai
Copy link
Copy Markdown
Contributor Author

I got it now. Before that commit, the current location was in OTD coordinates. After it, it fetches directly from the system, so the offset is no longer needed to translate the coordinates.

@gonX
Copy link
Copy Markdown
Member

gonX commented Oct 5, 2025

Thank you for investigating. Will approve given that statement.

@gonX gonX merged commit d173f25 into OpenTabletDriver:0.6.x Oct 5, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working desktop OpenTabletDriver.Desktop library, UX and Daemon use this as the core implementation. macos Affects the MacOS platform

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants