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

Ensure amount of scroll wheel rotations always captured accurately regardless of settings that affect size of scroll delta #64

Open
Inwerpsel opened this issue Mar 2, 2024 · 0 comments
Labels
good first issue Good for newcomers needs info The issue probably doesn't have enough information/ideas to be actionable at this time

Comments

@Inwerpsel
Copy link
Owner

Currently, a fixed step size of 100 is used to calculate the amount of steps (individual rotations of a stepped mouse scroll wheel) to capture during wheel events. This is used for scrolling over the history component and scrolling over the frame scale slider.

It produces an accurate result even when the event fires with a longer delta. This happens if the wheel is scrolled fast enough and/or a long running task delays the event.

This is probably the default step size on most devices, but I'm sure it can have another value depending on multiple factors in the environment.

Since the amount of steps is rounded to the nearest integer, step sizes between 50 and 100 should still have the same behavior when scrolling one at a time, and only when the delta comprises multiple steps would become somewhat slower.

Values below 50, however, will result in 0 steps because of rounding down.

Deltas above 150 would make it impossible to scroll one at a time.

Todo

  • Figure out common possible values for the delta and which factors affect its size
  • Test what happens when using continuous scrolling (unlocked scroll wheel)
  • Test what happens on alternative "scrolling" techniques (swiping on touch screens, custom scroll controls on a keyboard)
  • If necessary, add configuration for the step size. Could request user to scroll mouse a single time and capture
@Inwerpsel Inwerpsel added good first issue Good for newcomers needs info The issue probably doesn't have enough information/ideas to be actionable at this time labels Mar 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers needs info The issue probably doesn't have enough information/ideas to be actionable at this time
Projects
None yet
Development

No branches or pull requests

1 participant