fix: animating or removing marker depending on distance

Author: kikoso

library/src/main/java/com/google/maps/android/clustering/view/ClusterRendererMultipleItems.java

@@ -53,6 +53,7 @@
 import com.google.android.gms.maps.model.MarkerOptions;
 import com.google.maps.android.R;
 import com.google.maps.android.RendererLogger;
+import com.google.maps.android.SphericalUtil;
 import com.google.maps.android.clustering.Cluster;
 import com.google.maps.android.clustering.ClusterItem;
 import com.google.maps.android.clustering.ClusterManager;
@@ -438,6 +439,8 @@ private class RenderTask implements Runnable {
         private SphericalMercatorProjection mSphericalMercatorProjection;
         private float mMapZoom;
 
+        private static final double EPSILON = 1e-9;
+
         private RenderTask(Set<? extends Cluster<T>> clusters) {
             this.clusters = clusters;
         }
@@ -455,6 +458,10 @@ public void setMapZoom(float zoom) {
             this.mSphericalMercatorProjection = new SphericalMercatorProjection(256 * Math.pow(2, Math.min(zoom, mZoom)));
         }
 
+        private boolean areClose(double a, double b) {
+            return Math.abs(a - b) < EPSILON;
+        }
+
         @SuppressLint("NewApi")
         public void run() {
             final MarkerModifier markerModifier = new MarkerModifier();
@@ -532,9 +539,17 @@ public void run() {
                     final Point point = mSphericalMercatorProjection.toPoint(marker.position);
                     final Point closest = findClosestCluster(newClustersOnScreen, point);
                     if (closest != null) {
-                        markerModifier.remove(true, marker.marker);
-                        RendererLogger.d("ClusterRenderer", "Animating then removing marker at position: " + marker.position);
-                    } else if (mClusterMarkerCache.mCache.keySet().iterator().hasNext() && mClusterMarkerCache.mCache.keySet().iterator().next().getItems().contains(marker.clusterItem)) {
+                        LatLng animateTo = mSphericalMercatorProjection.toLatLng(closest);
+                        if (areClose(marker.position.latitude, animateTo.latitude) && areClose(marker.position.longitude, animateTo.longitude)) {
+                            // Treat them as the same, no need for animation
+                            markerModifier.remove(true, marker.marker);
+                            RendererLogger.d("ClusterRenderer", "Removing marker without animation (coordinates are very close)");
+                        } else {
+                            // If they are not close, proceed with the original logic
+                            markerModifier.animateThenRemove(marker, marker.position, animateTo);
+                        }
+                    }
+                    else if (mClusterMarkerCache.mCache.keySet().iterator().hasNext() && mClusterMarkerCache.mCache.keySet().iterator().next().getItems().contains(marker.clusterItem)) {
                         T foundItem = null;
                         for (Cluster<T> cluster : mClusterMarkerCache.mCache.keySet()) {
                             for (T clusterItem : cluster.getItems()) {