fix: zIndex for Clustering #1484
Conversation
kikoso
changed the title
| @@ -906,6 +910,13 @@ protected void onClusterItemUpdated(@NonNull T item, @NonNull Marker marker) { | |||
| protected void onBeforeClusterRendered(@NonNull Cluster<T> cluster, @NonNull MarkerOptions markerOptions) { | |||
| // TODO: consider adding anchor(.5, .5) (Individual markers will overlap more often) | |||
| markerOptions.icon(getDescriptorForCluster(cluster)); | |||
| Iterator<T> iterator = cluster.getItems().iterator(); | |||
There was a problem hiding this comment.
This code next more explanation. What is the goal of this code and how does it work?
It also strike me as very odd. I am not used to seeing an iterator w/o a loop. This has a strange code smell to it. If the intent is to only use the second element of the cluster, this seems like a very strange way to go about it.
There was a problem hiding this comment.
Cluster.getItems() returns Collection.
Since there are no methods like get in the Collection interface, I use iterator to get the first element.
I see another way to do this:
ArrayList<T> items = new ArrayList<>(cluster.getItems());
Do you think this option would be clearer?
There was a problem hiding this comment.
Hi @Programmeister .
That solutions sounds a bit better than the current iterator one IMO.
Programmeister
force-pushed
the
bugfix/1424
branch
from
d2658e3 to
ab01358
Compare
Programmeister
force-pushed
the
bugfix/1424
branch
from
ab01358 to
7c21890
Compare
…erRendered and onBeforeClusterItemRendered
Programmeister
force-pushed
the
bugfix/1424
branch
from
7c21890 to
723b46a
Compare
* feat: added ContinuousZoomEuclideanAlgorithm * feat: added ContinuousZoomEuclideanAlgorithm * feat: added ContinuousZoomEuclideanAlgorithm * feat: replaced methods in NonHierarchicalDistanceBasedAlgorithm.java * feat: refactored to create ContinuousZoomEuclideanCentroidAlgorithm.java
# [3.15.0](googlemaps/android-maps-utils@v3.14.0...v3.15.0) (2025-08-19) ### Features * added ContinuousZoomEuclideanCentroidAlgorithm ([googlemaps#1559](googlemaps#1559)) ([b781ff8](googlemaps@b781ff8))
|
Sorry. This PR got corrupted somehow beyond my ability to fix. I created a new PR with the original change here: #1566. Thank you for your contribution! |
Add zIndex to markerOptions in onBeforeClusterRendered and onBeforeClusterItemRendered
Fixes #1424