[PR #202] [MERGED] MacOS Multimonitor support #327

Closed
opened 2026-05-05 22:16:54 -06:00 by gitea-mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/feschber/lan-mouse/pull/202
Author: @Jacoby6000
Created: 9/18/2024
Status: Merged
Merged: 9/20/2024
Merged by: @feschber

Base: mainHead: macos-multi-monitor-support


📝 Commits (1)

📊 Changes

1 file changed (+80 additions, -13 deletions)

View changed files

📝 input-emulation/src/macos.rs (+80 -13)

📄 Description

I'm pretty new to rust, so don't be afraid to tell me anything I did horribly wrong 😅

This seems to fix multi-monitors on macos and hasn't had any issues from what I can tell.

The basic process is:

  • Get the current monitor that the mouse is on
  • Check what the new position would be if we just added dx and dy
  • Check if there is a display at that new position
    • If there is a different display there, clamp using its bounds
    • If there is not a different display there, clamp using the current monitor's bounds

Resolves #83


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/feschber/lan-mouse/pull/202 **Author:** [@Jacoby6000](https://github.com/Jacoby6000) **Created:** 9/18/2024 **Status:** ✅ Merged **Merged:** 9/20/2024 **Merged by:** [@feschber](https://github.com/feschber) **Base:** `main` ← **Head:** `macos-multi-monitor-support` --- ### 📝 Commits (1) - [`cb1f8b2`](https://github.com/feschber/lan-mouse/commit/cb1f8b276f5ebfa196c135021f1f67fcd7a07e00) Fix multimonitors ### 📊 Changes **1 file changed** (+80 additions, -13 deletions) <details> <summary>View changed files</summary> 📝 `input-emulation/src/macos.rs` (+80 -13) </details> ### 📄 Description I'm pretty new to rust, so don't be afraid to tell me anything I did horribly wrong 😅 This seems to fix multi-monitors on macos and hasn't had any issues from what I can tell. The basic process is: * Get the current monitor that the mouse is on * Check what the new position would be if we just added dx and dy * Check if there is a display at that new position * If there is a different display there, clamp using its bounds * If there is not a different display there, clamp using the current monitor's bounds Resolves #83 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
gitea-mirror 2026-05-05 22:16:54 -06:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/lan-mouse#327
No description provided.