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

Add mouse wheel setting #303

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

Add mouse wheel setting #303

wants to merge 3 commits into from

Conversation

ykszk
Copy link

@ykszk ykszk commented Apr 1, 2024

Currently, mouse wheel and ctrl-wheel are statically assigned to zooming in/out and moving to next/prev image respectively.
With this PR, users will be able to swap these assignments.

Changes

  • add a new persistent setting mouse_wheel_action
  • add Mouse section in the preferences UI for changing the setting (screenshot below)

mouse_wheel_setting

New behavior

  • With Zoom selected (default), mouse wheel works as zoom in/out and ctrl-wheel works as next/prev.
  • With Next/Prev selected, mouse wheel works as next/prev and ctrl-wheel works as zoom in/out.

Now mouse wheel can switch between:
(default) wheel: zoom and ctrl+wheel: next/prev
and
wheel: next/prev and ctrl+wheel: zoom
@woelper
Copy link
Owner

woelper commented Apr 5, 2024

Sorry for the delay!

First of all, thank you for this PR!

I really like this feature. I am thinking if we could solve this in a more generic way - for example could we maybe just expose wheel up and wheel down and have users set his directly, by setting shift + scroll to a specific action? What do you think?

@ykszk
Copy link
Author

ykszk commented Apr 10, 2024

Thanks for the comment! I implemented a better UI for the setting.

Screenshot_20240410_221130

In the screenshot, settings are changed from the default to showcase the various UI states, but the default behavior isn't changed.

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.

None yet

2 participants