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

Remove zoom mode #647

Closed
wants to merge 1 commit into from
Closed

Remove zoom mode #647

wants to merge 1 commit into from

Conversation

cmkohnen
Copy link
Collaborator

Since we have finer control over zooming now, this mode can be dropped imo.

Signed-off-by: Christoph Matthias Kohnen <[email protected]>
@sstendahl
Copy link
Owner

sstendahl commented Dec 13, 2023

Although there's indeed more ways to zoom now, the touchpad and scrolling gestures all do zoom equally in all directions, so it's not possible to zoom along one axis only. Personally, I think it can be very convenient to be able to draw a zoom box around a certain part of the plot somehow. Another issue is that the new zoom controls don't work on PC's if you don't have a mouse with a scroll wheel.

The same is true for panning by the way. Which can now also be accessed using the middle mouse button plus moving, but also there it's the case that this does require that you don't use a very basic mouse that only has two buttons. They're somewhat rare nowadays but still in use.

@cmkohnen
Copy link
Collaborator Author

Well personally I never really used this zoom mode. But if it is still useful, as you describe, I'm not at all against keeping it.

@cmkohnen cmkohnen closed this Dec 14, 2023
@sstendahl sstendahl deleted the remove-zoom-mode branch January 1, 2024 10:05
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.

2 participants