Skip to content

Conversation

@futtetennista
Copy link

No description provided.

@damianflannery
Copy link
Owner

Thanks, it looks good! Sticking with the builder pattern, I went the whole hog and augmented this to enable you to supply a custom ClusterSpot and configure the text displayed on the cluster (e.g. in case you wanted to show 'H' for high instead of the number of markers contained), text colour, text size etc.

I also fixed the code to properly scale the marker to the custom drawable (e.g. before if you replaced the cluster spot background with one of the purple map markers it will stretch it to fit the bounds defined in resources e.g. 32 x 32).

What's the best way to iterate... should I commit to this branch or should I fork this and provide you a patch (Inception :) )?

I still need to add javadocs but it would be cool if you could take a look first.

damianflannery added a commit that referenced this pull request Nov 6, 2012
Added possibility to supply custom cluster drawables
@damianflannery damianflannery merged commit 160a900 into damianflannery:clustering Nov 6, 2012
@damianflannery
Copy link
Owner

In the end I just powered through, merged and then committed my changes to the clustering branch. Thanks for the contribution.

@futtetennista
Copy link
Author

Well...thank you! Haven't done that much really. I'm looking forward to using the library!

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.

2 participants