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

Zoom to cluster by extent #4

Open
lonvia opened this issue Aug 31, 2021 · 1 comment
Open

Zoom to cluster by extent #4

lonvia opened this issue Aug 31, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@lonvia
Copy link
Member

lonvia commented Aug 31, 2021

Clicking on a cluster to zoom in is a cool feature. Frequently it works well but just as often it doesn't zoom in enough. If I understand the code right, then it zooms in to the zoom level given by the cluster. I wonder if it would work better if the cluster would communicate the bounding box for of the features it contains. The map could then zoom directly to this extent.

@AntoJvlt AntoJvlt added the enhancement New feature or request label Sep 1, 2021
@AntoJvlt
Copy link
Collaborator

AntoJvlt commented Sep 1, 2021

Indeed this could produce a way more accurate zoom and better user experience. However this needs some work since the extent should be added to each cluster while generating them on the server side. Currently we are adding the clusterExpansionZoom manually in clustering-vt, but for the extent we should instead do some processing by using the getLeaves() method of supercluster.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants