-
Notifications
You must be signed in to change notification settings - Fork 2
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
small map changes #676
small map changes #676
Conversation
to avoid occational overflows
and no zoom out on reset
✔️ Preview link ready! 🔨 Explore the source changes: 25237a8 🔍 Inspect the deploy logs: https://app.netlify.com/sites/cmu-delphi-covidcast/deploys/5fc92cf8c145720007e9f806 😎 Browse the preview: https://deploy-preview-676--cmu-delphi-covidcast.netlify.app |
Preview link ready! Built with commit 14b4e4f https://deploy-preview-676--cmu-delphi-covidcast.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I didn't notice a big difference in how far you could zoom out, but the level to which you can zoom in looks like an improvement. I also like the automatic focus changes.
LGTM for the zoom-in limit Also, not seeing the zoom-out limit, thank for sharing @sgratzl |
Prerequisites:
dev
branchdev
Summary
various changes related to the map