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

Zooming out doesn't cause a cluster to get formed. #2

Closed
MrHus opened this issue Oct 30, 2012 · 3 comments
Closed

Zooming out doesn't cause a cluster to get formed. #2

MrHus opened this issue Oct 30, 2012 · 3 comments

Comments

@MrHus
Copy link

MrHus commented Oct 30, 2012

It would be nice to add clusters for multiple zoom levels.

For instance while zooming out to the whole world create a cluster on europe and afrika.
And zooming in on country level see France and Belgium and Holland.

Imgur

@demoend
Copy link

demoend commented Nov 12, 2012

I agree. The clustering should be determined by the zoom level. Perhaps a threshold is set to determine how close the pins can be before they cluster into one pin. The example app does not handle zooming out well; The group of annotations no longer cluster at a low zoom. I expect the clustering to work more like the Apple Photo app at any zoom level.

@Ecco
Copy link
Contributor

Ecco commented Nov 23, 2012

Zoom "levels" used to make sense when using Google Maps on iOS, since those were using bitmap tiles. Note that MapKit never exposed a zoom level API though.

Now on iOS 6, maps are using vector-based drawing. So it's not even clear what a zoom level would be.

The current algorithm is set to work independently of the location of the actual pins : it'll cluster them up until they match an upper limit (40 in this demo). So even if you zoom out a lot, you'll always have 40 cluster pins on the map, even if they end up being close one to each other.

What you'd like to add is to lower this limit if said clusters are too close to each other. This would indeed make sense, and I'm opening another ticket for that.

@Ecco Ecco closed this as completed Nov 23, 2012
@Ecco
Copy link
Contributor

Ecco commented Nov 23, 2012

See issue #8

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

No branches or pull requests

3 participants