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.

Sign up for GitHub

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

Jump to bottom

Remove zoom to geom on click functionality #55

Open
4lm opened this issue Sep 26, 2022 · 2 comments
Open

Remove zoom to geom on click functionality #55

4lm opened this issue Sep 26, 2022 · 2 comments
Assignees
@henhuy
Labels
enhancement 🚀 New feature or request wontfix 💔 This will not be worked on

Comments

@4lm
Copy link
Contributor

4lm commented Sep 26, 2022

While #54 happened I also realized, that the zoom to geom on click functionality is still in there and is canceled out by the popup normally. Can IMO be removed. Clean Code!
@4lm 4lm added the enhancement 🚀 New feature or request label Sep 26, 2022
@4lm 4lm self-assigned this Sep 26, 2022
@henhuy
Copy link
Collaborator

henhuy commented Sep 29, 2022

You mean zooming only via mouse wheel or +/- buttons? Not when clicking on a region, right?
As every region gets a popup, this would lead to disturbing behaviour - so it's good to remove this feature.

@4lm
Copy link
Contributor Author

4lm commented Sep 29, 2022

You mean zooming only via mouse wheel or +/- buttons? Not when clicking on a region, right?

Yes

As every region gets a popup, this would lead to disturbing behaviour - so it's good to remove this feature.

Yes. That's what I meant.

@henhuy henhuy unassigned 4lm Mar 9, 2023
@4lm 4lm self-assigned this Apr 3, 2023
@henhuy henhuy assigned henhuy and unassigned 4lm Jul 3, 2023
@nesnoj nesnoj added the wontfix 💔 This will not be worked on label Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@henhuy henhuy

Labels
enhancement 🚀 New feature or request wontfix 💔 This will not be worked on
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@4lm @nesnoj @henhuy