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

Make Scroll Wheel velocity More Configurable #290

Closed
Dwaynekj opened this issue Aug 23, 2016 · 6 comments
Closed

Make Scroll Wheel velocity More Configurable #290

Dwaynekj opened this issue Aug 23, 2016 · 6 comments

Comments

@Dwaynekj
Copy link
Contributor

Dwaynekj commented Aug 23, 2016

See discussion here #282

@Dwaynekj
Copy link
Contributor Author

@hugoeanogueira
Copy link
Contributor

hugoeanogueira commented Aug 25, 2017

Hey! I've been doing some work to make mousewheel scrolling speed more configurable :p
I've tracked the feature to finbars dependency (didn't see anything being triggered in ThumbwheelScrolling.js).

Two new properties were added scrollingHorizontalMouseWheelMultiplier and scrollingVerticalMouseWheelMultiplier, which are proxied to finbars.

These values can be set as:

  • between 0 and 1 -> decrease speed;
  • 1 -> original speed (default value);
  • above 1 -> increase speed;
  • negative values will invert the scrolling (could probably apply a modulo on this?...)

I'll open pull requests for both repositories for you to test and comment.
Thanks!

@hugoeanogueira
Copy link
Contributor

The PRs:

@Dwaynekj
Copy link
Contributor Author

@hugoeanogueira Did you test on Windows and Mac? Additionally, any thoughts into delegating the event from the ThumbwheelScrolling feature chain as opposed to from the finbars.

@hugoeanogueira
Copy link
Contributor

Not yet, I need to get a Windows machine first and do some testing.
I've tried to do some debugging in ThumbwheelScrolling.js, but haven't managed to see any action in there. Maybe I have to configure anything for the event to be propagated to the ThumbwheelScrolling feature chain?

@joneit
Copy link
Contributor

joneit commented May 24, 2018

Resolved by Hypergrid v2.1.11 (PR #722).

@joneit joneit closed this as completed May 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants