Skip to content

MOUSE_MOVE_RELATIVE flag fix#3

Merged
kennyhml merged 3 commits intokennyhml:masterfrom
dvdco:master
Jun 1, 2023
Merged

MOUSE_MOVE_RELATIVE flag fix#3
kennyhml merged 3 commits intokennyhml:masterfrom
dvdco:master

Conversation

@dvdco
Copy link
Contributor

@dvdco dvdco commented Jun 1, 2023

corrected the MouseFlag.MOUSE_MOVE_RELATIVE in _consts.py updated the inputs.py to use real relative move, rather than getting the mouse position, add delta and abs move now can work better in fps games

precisely:

  • Commit message: Fix mouse relative movement error
  • This commit changes two files, namely "_consts.py" and "inputs.py". In "_consts.py", the value of the "MOUSE_MOVE_RELATIVE" variable was changed to "0x000" to fix the mouse relative movement error. Additionally, in "inputs.py", the incorrect implementation of "move_relative" was fixed, and the "@requires_driver" decorator was added to ensure that the necessary driver is present for the operation. Also, the scrolling direction error in the "scroll" function was fixed, and the Mouse Move flag "MOUSE_MOVE_RELATIVE" was added to ensure accuracy. Finally, a newline character was added at the end of the "key_down" function.

corrected the MouseFlag.MOUSE_MOVE_RELATIVE in _consts.py
updated the inputs.py to use real relative move, rather than getting the mouse position, add delta and abs move
now can work better in fps games
@kennyhml kennyhml changed the title Add files via upload MOUSE_MOVE_RELATIVE flag fix Jun 1, 2023

stroke = MouseStroke(MouseState.MOUSE_WHEEL, 0, amount, 0, 0, 0)
stroke = MouseStroke(
MouseState.MOUSE_WHEEL, MouseFlag.MOUSE_MOVE_RELATIVE, amount, 0, 0, 0
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets keep this as 0 here because MOUSE_MOVE_RELATIVE just adds to the confusion when the function is not intended to move the cursor,

@kennyhml kennyhml merged commit 61ac50c into kennyhml:master Jun 1, 2023
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

Successfully merging this pull request may close these issues.

2 participants