Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moving mouse positions up the screen not possible. #83

Closed
dooplicate opened this issue Jan 25, 2022 · 4 comments
Closed

Moving mouse positions up the screen not possible. #83

dooplicate opened this issue Jan 25, 2022 · 4 comments

Comments

@dooplicate
Copy link

dooplicate commented Jan 25, 2022

I do not see how it is possible to move a mouse up the screen with assert_count() disabling negative numbers by way of stopifnot(x >= 0) via moveMouseTo(). Therefore making any mouse movement a one-way proposition. Unless there is a solution I am not aware of this appears to be a major issue.

@gaborcsardi
Copy link
Collaborator

gaborcsardi commented Jan 25, 2022

This is probably a bug and instead of assert_count() we should just check that the number is an integer. Do you want to submit a pull request for this? (No pressure.)

@dooplicate
Copy link
Author

I agree, or making assert_count less zealous. You are much better qualified though so I'll leave this to you sir, thanks! Looking forward to a fix.

@gaborcsardi
Copy link
Collaborator

Can you try the version that I just pushed?

@dooplicate
Copy link
Author

dooplicate commented Jan 25, 2022

Everything is working as intended now. Thank you for the hasty reply!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants