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

[FEATURE] Add catch-all MapEventZoom #1732

Open
opk12 opened this issue Nov 13, 2023 · 0 comments
Open

[FEATURE] Add catch-all MapEventZoom #1732

opk12 opened this issue Nov 13, 2023 · 0 comments
Assignees
@josxha
Labels
feature This issue requests a new feature P: 3 (low) (Default priority for feature requests) S: core Scoped to the core flutter_map functionality

Comments

@opk12
Copy link

opk12 commented Nov 13, 2023 •
edited
Loading

What do you want implemented?

A generic "zoom" event, to simplify updating the internal state of the app that depends on the zoom level number.

What other alternatives are available?

Currently the app must enumerate all the gestures: zoom in and out buttons, MapEventDoubleTapZoom, MapEventScrollWheelZoom.

This approach

  • does not handle the pinch zoom event; I'm told that there is a multi finger event that is redirected into a MapEventMove, but this is easy to miss in the API
  • ignores future gestures: the two-finger tap requested in [FEATURE] Two fingers tap to zoom out #1432
  • is verbose

Can you provide any other information?

No response

Severity

Annoying: Currently have to use workarounds
@opk12 opk12 added feature This issue requests a new feature P: 3 (low) (Default priority for feature requests) labels Nov 13, 2023
@opk12 opk12 mentioned this issue Nov 13, 2023
Closed
@josxha josxha mentioned this issue Nov 19, 2023
Open
@josxha josxha added this to the v7.0 milestone Dec 2, 2023
@josxha josxha mentioned this issue Dec 10, 2023
Closed
38 tasks
@JaffaKetchup JaffaKetchup added the S: core Scoped to the core flutter_map functionality label Dec 10, 2023
@JaffaKetchup JaffaKetchup changed the title [FEATURE] API: Zoom event [FEATURE] Add catch-all MapEventZoom Dec 10, 2023
@josxha josxha self-assigned this Dec 13, 2023
@josxha josxha mentioned this issue Dec 19, 2023
Open
@josxha josxha moved this to To do in Development Jan 21, 2024
@josxha josxha removed this from the v7.0 milestone Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@josxha josxha

Labels
feature This issue requests a new feature P: 3 (low) (Default priority for feature requests) S: core Scoped to the core flutter_map functionality
Projects
Development
Status: To do
Milestone
No milestone
Development

No branches or pull requests
3 participants
@josxha @opk12 @JaffaKetchup