Skip to content
This repository was archived by the owner on Apr 27, 2024. It is now read-only.

Conversation

@stleusc
Copy link
Contributor

@stleusc stleusc commented May 6, 2014

Feature to switch double-tap zoom to either zoom around tap location (currently like that) or zoom around center of map, keeping this centered!
I need to be able to switch the two modes while the map is showing.

Another request that kinda fits in here (though not double tap, but still zoom)
Request
Feature to allow to disable zoom & drag map. Meaning
a) allow 2 finger zoom AND drag/move map at the same time
b) allow 2 finger zoom but dragging both fingers to move map will not move it

Good example is current Google Maps app.
for a) move map around and then do 2 finger zoom and then drag both fingers to move map
for b) center map on you and THEN do the above and you will see that the map does not move

Same procedure can be used to test the double tap zoom btw....

@stleusc stleusc changed the title Update MapEventLayer.java to fix #62 align zoom behavior to Gmaps when having map centered around location May 6, 2014
@hjanetzek
Copy link
Member

So basically you want a mode where the user cannot move the map by touch gestures, right? Maybe one could do this in one place. I've commited some functions to ViewController to temporarily disable moveMap() - No time to test atm though :) Could you give it a try? 328132d

@stleusc
Copy link
Contributor Author

stleusc commented May 6, 2014

No that is not what I want. I want the gmaps behavior!
I have a button to enable 'follow-me' mode, where the map centers user location.
In that mode (also in gmaps) the map does not move while zooming! But of course you can just pan it around and turn of 'follow-me' with that.
For the double tap zoom the map will keep the center fixed when 'follow-me' is active, otherwise it keeps the touch location fixed.

If that does not make sense, open gmaps and drag/zoom or double tap, then center on your location (button lower right corner) and try again!

@stleusc
Copy link
Contributor Author

stleusc commented May 6, 2014

@hjanetzek nope, that commit would not do what I need at all :)

@stleusc
Copy link
Contributor Author

stleusc commented May 6, 2014

@hjanetzek my commit does exactly what I need. Check it out, only few changes and that might explain what it is about.

@hjanetzek
Copy link
Member

I see. Slightly refactored version of yours, not yet tested: 1d869b8

@stleusc
Copy link
Contributor Author

stleusc commented May 7, 2014

works as expected!
you can merge it into master...

@hjanetzek
Copy link
Member

Merged. Thanks for the patch!

@hjanetzek hjanetzek closed this May 12, 2014
hjanetzek pushed a commit to hjanetzek/vtm that referenced this pull request Nov 18, 2018
hjanetzek pushed a commit to hjanetzek/vtm that referenced this pull request Nov 18, 2018
hjanetzek pushed a commit to hjanetzek/vtm that referenced this pull request Nov 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants