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

Zoom level indicator auto-update #243

Closed
opk12 opened this issue Nov 10, 2023 · 6 comments
Closed

Zoom level indicator auto-update #243

opk12 opened this issue Nov 10, 2023 · 6 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@opk12
Copy link

opk12 commented Nov 10, 2023

Do I understand correctly that the number at the bottom right is the zoom level? It does not change, when I pinch the slippy map to zoom in or zoom out. It should auto-update instead. But then I press the blue arrow at the bottom right (toggle menu bar icons) and it gets updated.

@moovida moovida self-assigned this Nov 12, 2023
@moovida moovida added the bug Something isn't working label Nov 12, 2023
@moovida moovida added this to the 1.7.x milestone Nov 12, 2023
@moovida
Copy link
Member

moovida commented Nov 12, 2023

Hi @opk12 , yes it is the zoomlevel. Up to some time ago it was not possible in the flutter map lib to catch the pinch zoom event, so the number it connected to the zoom in and out buttons.
I just saw that now some more events:

  • MapEventDoubleTapZoom
  • MapEventScrollWheelZoom

but still no pinch event.
Adding these in the meanwhile to fix at least the double tap and the wheel scroll for desktop versions.

@opk12
Copy link
Author

opk12 commented Nov 13, 2023

Thank you. Is it fleaflet/flutter_map? If so, I opened this ticket for a generic zoom event API, which seems not to exist indeed in the class list, if it can help somehow

@moovida
Copy link
Member

moovida commented Nov 22, 2023

Great, thanks. When the event will be implemented, we can also move it to smash.

@moovida
Copy link
Member

moovida commented Nov 23, 2023

Hi @opk12 , for a different reason I ended up checking the MapEvents once more and found that there is a multi finger event that is redirected into a MapEventMove. So listening to that one made the zoom indicator finally work. I want to do some more testing on this, but seems to work nicely. Closing this for now.

@moovida moovida closed this as completed Nov 23, 2023
@opk12
Copy link
Author

opk12 commented Nov 24, 2023

Thank you. FYI There's a gesture API restructuring in the works. fleaflet/flutter_map#1715

@moovida
Copy link
Member

moovida commented Nov 25, 2023

Yep, thanks, I saw this and am not looking forward to migrate to flutter_map 5. Already 4 has been a pain. :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants