android-maps-utils icon indicating copy to clipboard operation
android-maps-utils copied to clipboard

Add utility for animating marker movement

Open barbeau opened this issue 5 years ago • 5 comments

Is your feature request related to a problem? Please describe. Right now the clustering algorithm animates markers moving into and out of clusters, but a general utility to animate marker movements on the map isn't exposed as an API in this library.

Animating marker movement is useful beyond just clustering - for example, I wanted to animate the movement of a vehicle marker rather than have it jump from one position to another.

Here's the existing clustering implementation of marker animations in this library: https://github.com/googlemaps/android-maps-utils/blob/db41dfa8d03f458edf2cccc4824f30713a232c54/library/src/main/java/com/google/maps/android/clustering/view/DefaultClusterRenderer.java#L1094

Describe the solution you'd like A few years back I implemented a general-use marker animation utility for an app I maintain, partially based on code from this project and https://gist.github.com/broady/6314689. The implementation is here: https://github.com/OneBusAway/onebusaway-android/blob/8eeda8586a394d293b3ae8bb99ef5c6df7a93176/onebusaway-android/src/google/java/org/onebusaway/android/map/googlemapsv2/AnimationUtil.java

I'd like to contribute the utility to this project, and potentially merge it with the implementation used in clustering so the code is shared where it makes sense.

Describe alternatives you've considered Do nothing - leave the clustering marker animation implementation private

barbeau avatar Sep 03 '20 20:09 barbeau

This issue has been automatically marked as stale because it has not had recent activity. Please comment here if it is still valid so that we can reprioritize. Thank you!

stale[bot] avatar Jan 03 '21 06:01 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. Please comment here if it is still valid so that we can reprioritize. Thank you!

stale[bot] avatar Jun 02 '21 16:06 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. Please comment here if it is still valid so that we can reprioritize. Thank you!

stale[bot] avatar Oct 02 '21 00:10 stale[bot]

Closing this. Please reopen if you believe it should be addressed. Thank you for your contribution.

stale[bot] avatar Apr 16 '22 04:04 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. Please comment here if it is still valid so that we can reprioritize. Thank you!

stale[bot] avatar Sep 21 '22 02:09 stale[bot]

Hi @barbeau,

Would it make sense to add the AnimationUtil class you developed before as a standalone class? I can add it on a PR for you with some tests to review.

kikoso avatar Jan 23 '23 14:01 kikoso

@kikoso yes, thanks, I think that would be great!

barbeau avatar Jan 23 '23 16:01 barbeau

I will work a bit on samples and test on this PR. Thanks @barbeau

kikoso avatar Jan 23 '23 16:01 kikoso

There is a parallel feature request since 2016 with 13 +1s in the public issue tracker for Maps SDK for Android. https://issuetracker.google.com/issues/35829563

wangela avatar Feb 09 '23 06:02 wangela

Thank you @barbeau for suggesting this feature and @kikoso for implementing it!

wangela avatar Feb 09 '23 06:02 wangela

:tada: This issue has been resolved in version 3.4.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

googlemaps-bot avatar Feb 09 '23 06:02 googlemaps-bot