Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Rework zoomIn animation to use ValueAnimators #11380

Closed
LukasPaczos opened this issue Mar 5, 2018 · 1 comment
Closed

Rework zoomIn animation to use ValueAnimators #11380

LukasPaczos opened this issue Mar 5, 2018 · 1 comment
Assignees
Labels
Android Mapbox Maps SDK for Android

Comments

@LukasPaczos
Copy link
Member

Currently, we are internally using core animation with MapView#setZoom(double zoom, PointF focalPoint, long duration) to gradually change zoom with double-tap gesture, two-tap gesture, and zoom controls. This animation is being interrupted by MapboxMap#moveCamera() that is a preferred way of building external camera animations (after removal of the previous API in #11201).

To support external animations without interrupting internal ones, when not explicitly ordered, we should rework Transform#zoomIn() and Transform#zoomOut() animations to use ValueAnimators, as do the rest of gesture animators.

@LukasPaczos LukasPaczos added the Android Mapbox Maps SDK for Android label Mar 5, 2018
@LukasPaczos LukasPaczos added this to the android-v6.0.0 milestone Mar 5, 2018
@LukasPaczos LukasPaczos self-assigned this Mar 5, 2018
@LukasPaczos
Copy link
Member Author

Landed with #11382.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Development

No branches or pull requests

1 participant