-
Notifications
You must be signed in to change notification settings - Fork 1k
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
made a minor change to _zoomOrSpiderfy to Spiderfy without zooming #415
Conversation
… everything in the cluster is at the same lat long. This prevents zooming into max level if there is no real reason to do so, since all points will be at the same location. This minor change increased usability in a personal project.
Travis picked up a couple of white space errors, fix those up and we'll bring this in :) |
Thanks, I fixed them, i only changed it on the main source file, not anywhere else, no need to change anywhere else correct? |
Yep that's the only file you should change. |
Okay made the change as a note to the new behavior in the option. :) |
made a minor change to _zoomOrSpiderfy to Spiderfy without zooming
Cool thanks! |
This is really helpful @rdenniston! Thanks for putting it together. |
If everything in the cluster is at the same lat long. This prevents zooming into max level if there is no real reason to do so, since all points will be at the same location. This minor change increased usability in a personal project.
Thought you guys might appreciate it. This is my first pull request ever, so if I am doing this totally wrong, please let me know.