-
-
Notifications
You must be signed in to change notification settings - Fork 6k
IE10 touch support #1076
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
IE10 touch support #1076
Conversation
…DomEvent.DoubleTap). DoubleTap touch on IE10 doesn't currently work but everything else does. Refs Leaflet#871
… count manually and clone the object so we can change the type.
…sues. Also fixes removeDoubleTapListener for all platforms
…them (cherry picked from commit ec2989e)
…ERANCE (cherry picked from commit b7e84ae)
…fter the drag has finished (clicking during a drag seems weird).
…g off screen bug without needing the work around
…document so that we get all of them, otherwise we will miss some and mess up our touch coun tracking.
…t and never let go otherwise we can miss MSPointerUp events.
…(which it of course isn't!)
| 'core/Handler.js', | ||
| 'map/handler/Map.TouchZoom.js'], | ||
| deps: ['MapAnimationZoom'], | ||
| desc: 'Enables smooth touch zooming on iOS and double tap on iOS/Android.' | ||
| desc: 'Enables smooth touch zooming on iOS and IE10 and double tap on iOS/IE10/Android.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Btw, smooth zoom is now on Android 4 too, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Multi-touch zoom is on android 4 yeah, we added it in 0.4:
https://github.com/CloudMade/Leaflet/blob/master/CHANGELOG.md#notable-new-features
This can indeed be tidied up! :)
Thanks to @oliverheilig and @veproza for testing and helping.
Fixes #871.
Hopefully doesn't break too many things ;-)