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

Handling continuous scroll key input. #33

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

unisons
Copy link

@unisons unisons commented Aug 29, 2018

Add impulse checking the current velocity value.
Without this patch, keeping pressing scroll key, it doessn't stop
properly. This patch make to set a upper limit to velocity value.

Add a function for distance instead of impluse. It will help to move a
specific lines. But it will ignore the air_drag value. It is hard to
calculate the impulse value to move a certain distance with air_drag.

unisons and others added 2 commits August 29, 2018 18:56
Add impulse checking the current velocity value.
Without this patch, keeping pressing scroll key, it doessn't stop
properly. This patch make to set a upper limit to velocity value.

Add a function for distance instead of impluse. It will help to move a
specific lines. But it will ignore the air_drag value. It is hard to
calculate the impulse value to move a certain distance with air_drag.
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.

1 participant