Skip to content
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

feat: implementation of diff #1438

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

feat: implementation of diff #1438

wants to merge 8 commits into from

Conversation

kikoso
Copy link
Collaborator

@kikoso kikoso commented Dec 10, 2024

The following PR implements the diff option to add, remove and update items on a Cluster. It also provides a sample on how to implement it on the app, including modifying the ClusterItem extended object to add the equals() method. It provides a Custom Renderer as a sample, as well.

Summary of changes:

  • Implementation of a diff() method in the ClusterManager, to pass at once all the required ClusterItems to be removed.
  • Sample implementing equals in the demo, to determine how the ClusterItems are rendered unique.
  • New Cluster to render multiple items.
  • New demo, to showcase how to use it

This is a diagram that showcases the relationship between the main components in the Clustering.

Screenshot 2024-12-13 at 11 19 44

@kikoso kikoso marked this pull request as draft December 10, 2024 13:42
@googlemaps-bot
Copy link
Contributor

googlemaps-bot commented Dec 10, 2024

Code Coverage

Overall Project 34.83% -8.95% 🍏
Files changed 0%

File Coverage
ClusterManager.java 0% -10.21%
ClusterRendererMultipleItems.java 0%
DefaultClusterRenderer.java 0% -2%

@kikoso kikoso marked this pull request as ready for review December 13, 2024 10:25
@kikoso kikoso added the snippet-bot:force-run Force snippet-bot runs its logic label Dec 17, 2024
@snippet-bot snippet-bot bot removed the snippet-bot:force-run Force snippet-bot runs its logic label Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants