fix multi-monitor not working in windows with moveMouseSmooth pipeline fixed #611
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…elegant solution and there a few small issues with mouse accelleration due to crude maths. however this will properly get the virtual screen size and map the coordinates calculated in moveMouseSmoothly then pass them to moveMouse pre calculated, with a flag to skip this step in mouseMove. TODO: merge these two functions and call an overload from node to streamline things a bit more
…he most elegant solution and there a few small issues with mouse accelleration due to crude maths. however this will properly get the virtual screen size and map the coordinates calculated in moveMouseSmoothly then pass them to moveMouse pre calculated, with a flag to skip this step in mouseMove. TODO: merge these two functions and call an overload from node to streamline things a bit more" This reverts commit 7b1c5fb.
…support. properly grab virtualdesk size not just primary display size. add a bit of distance based speed modifier for this as well.
Pull current verison
|
Hello @octalmage , could you please restart the AppVeyor build? I don't see a difference between "Environment: nodejs_version=Stable; Platform: x86" and "Environment: nodejs_version=12; Platform: x86". |
src/mouse.c
Outdated
|
|
||
| INPUT mouseInput = {0}; | ||
| mouseInput.type = INPUT_MOUSE; | ||
| mouseInput.mi.dx = x; | ||
| mouseInput.mi.dx = x - 100; |
Feat Mac OS Multi-Screen Support of captureScreen()
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello,
I've fixed the build problems for Pull Request #536 . The mouth smooth mouse move is working on for Linux and Windows for multiple screen. Unfortunately it works not under iOS.