-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Lock rotation when pinch-zooming #813
Comments
I had noted this in #553 (comment) (comment). Touch events on Android are very hard due to the poor API Google have designed as you have to write pretty much everything from scratch. Would love any touch experts (perhaps from JS side) to provide advice on this too. |
@ljbade, if you've got an Apple Developer membership, you might want to take a look at the video for the 2010 WWDC presentation called Advanced Gesture Recognition. It talks about how they implemented their gesture recognition system and the reasoning behind some of the design choices. |
@wsnook, cool I watch when I get to SF. For some reason United block *.mov files on their WiFi. |
Thanks @wsnook for the insight — didn't realize they ever talked about these internals. |
Hmm I do like their design w.r.t. to gesture states and the interaction between different gesture recognisers. Much better than the stock Android model. I am wonder if we should implement something that operates like the Apple design on Android. |
We should look high & low for any efforts to do this already outside of Google Play, then consider building it out ourselves as a standalone. IMHO it's something needed by top-notch Android apps and should exist. |
@bleege I see you use the same touch lib in the raster SDK. Did you ever search for other Android touch libs? |
@ljbade Here's the background on the Rotation Library decision in Android Raster: https://github.com/mapbox/mapbox-android-sdk/issues/525 |
On Android, it is very easy to change the rotation of the map when zooming. iOS has some mechanism to prevent that. We should have the same behavior on Android too (Google Maps does the same).
The text was updated successfully, but these errors were encountered: