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

Support panning with the mouse wheel. #95

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

giomasce
Copy link

The movement scheme is based on common browsers, like Firefox and Chromium, assumed to be a good approximation of what users find intuitive.

I realize that would be a change WRT the current behavior, and honestly I'm not too attached to it. I can switch to any other combination for the modifier keys if that's preferred.

The movement scheme is based on common browsers, like Firefox
and Chromium, assumed to be a good approximation of what users
find intuitive.
@mikesart
Copy link
Owner

@Plagman Do you have any thoughts on this one Pierre-Loup? Thanks!

@Plagman
Copy link
Collaborator

Plagman commented Nov 16, 2024

Not convinced it makes sense to change as the default, given you're often constantly scrolling in X/Y directions and need quick access to zoom in/out as well. I think the current model matches well with usage, and would be hard to change from under people who are used to it. Might make sense to have as an option, or with a modifier key?

@giomasce
Copy link
Author

Not convinced it makes sense to change as the default, given you're often constantly scrolling in X/Y directions and need quick access to zoom in/out as well. I think the current model matches well with usage, and would be hard to change from under people who are used to it. Might make sense to have as an option, or with a modifier key?

I can probably adapt to any other modifier combination. What would your favorite one be? Basically you have to choose a modifier (including no modifier) for each of zoom, pan horizontally and pan vertically.

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.

3 participants