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

[Windows] Scrollbar sensitivity seems very low #95

Open
Pokechu22 opened this issue Jun 24, 2020 · 9 comments
Open

[Windows] Scrollbar sensitivity seems very low #95

Pokechu22 opened this issue Jun 24, 2020 · 9 comments
Labels
widgetry Improve our internal UI library

Comments

@Pokechu22
Copy link

Sensitivity on scrollbars seems to be very low (it takes a lot of scrolling to go down one line). This is specifically for scrollbars on UIs; ingame, scrolling to zoom in or out seems to be at a good speed.

Version 0.2.0d.
Mouse: Microsoft Wireless Mobile Mouse 4000, using mouse and keyboard center
Display resolution: 1366 x 768 (small enough that a lot of UIs need scroll bars), should be possible to also run into this in windowed mode.

@dabreegster
Copy link
Collaborator

https://github.com/dabreegster/abstreet/blob/626636ece77ebc8c77fdc42e5a19b1b2eca80fc1/ezgui/src/event.rs#L66

It's probably that. I made up this constant a while ago on one particular Mac; I don't have regular access to one. I'll try out a fix and send a link to test

@dabreegster
Copy link
Collaborator

@Pokechu22
Copy link
Author

Ah, yes, Mac scrolling. I have a bit of experience with it; basically the acceleration on it is insane and the easiest (and only) workaround I know of for it is to just ignore the scroll delta values and just use a constant offset for each scroll event.

That said, that build doesn't seem to have made a difference for me on Windows; in both that build and the regular one, it takes about 7 scrolls to scroll from the top of the settings menu (on my resolution, that has "scale factor for text" as the lowest visible thing) to the bottom where the apply button is. I think my mouse is still triggering the LineDelta case.

@dabreegster
Copy link
Collaborator

That writeup is really thorough! If you're hitting the LineDelta case, then I'm not sure what to do other than expose a setting for you to tune the speed. I still need to add a little area in the settings panel to test that scrolling, and start saving settings between sessions.

@dabreegster
Copy link
Collaborator

I think #310 might've improved this. Would you mind testing again from HEAD, or from the release that I'll upload in a few hours?

@Pokechu22
Copy link
Author

It doesn't seem to be any different between v0.2.8 and v0.2.10 for me, though I do see that there's a "Scroll speed for menus" option (changing it to 20 from 5 feels better for me). Note that I'm running on Windows.

@dabreegster dabreegster changed the title Scrollbar sensitivity seems very low [Windows] Scrollbar sensitivity seems very low Sep 9, 2020
@michaelkirk
Copy link
Collaborator

@Pokechu22 - If I prepared a debug branch to get more scroll-specific information, would you be willing to run it and share the log?

@Pokechu22
Copy link
Author

Sure.

@michaelkirk
Copy link
Collaborator

michaelkirk commented Sep 10, 2020

Ok @Pokechu22 - here's what I'd like you to do:

  1. checkout out the branch from my fork: https://github.com/michaelkirk/abstreet/tree/mkirk/scroll-debug and run the widgetry demo
git remote add michaelkirk https://github.com/michaelkirk/abstreet.git
git fetch michaelkirk
git checkout mkirk/scroll-debug
cargo run --example demo > log.out
  1. Do a scroll lap around the scrollable widget as shown here:
    widgetry_scroll_demo mov

  2. Upload the contents of the logging in log.out to https://gist.github.com (or some other pastebin)

  3. Send the link to the uploaded load

Thanks for any help in figuring this out!

@dabreegster dabreegster added widgetry Improve our internal UI library and removed platform specific bug labels Feb 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
widgetry Improve our internal UI library
Projects
None yet
Development

No branches or pull requests

3 participants