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

depending on map pan, lng from leaflet can be out of range #571

Closed
matthewberryman opened this issue Aug 5, 2018 · 5 comments
Closed

depending on map pan, lng from leaflet can be out of range #571

matthewberryman opened this issue Aug 5, 2018 · 5 comments
Assignees
Labels

Comments

@matthewberryman
Copy link
Contributor

No description provided.

@matthewberryman matthewberryman self-assigned this Aug 5, 2018
@matthewberryman matthewberryman changed the title depending on map pan, lng and lat from leaflet can be out of range depending on map pan, lng from leaflet can be out of range Aug 6, 2018
matthewberryman added a commit that referenced this issue Dec 3, 2018
@matthewberryman
Copy link
Contributor Author

fixed with aa2fb1a 54579a5

@LucieGueuning
Copy link

LucieGueuning commented Dec 5, 2018

image
Still happening.

@LucieGueuning LucieGueuning reopened this Dec 5, 2018
@matthewberryman
Copy link
Contributor Author

@LucieGueuning I have repeatedly tested on Chrome and cannot reproduce this issue on zooming out, noting that we start with a sensible default zoom and also preserve the current zoom level and location across page loads.
screenshot 2018-12-06 09 46 00

Can you please reset cookies and cache your end to make sure it's picked up the latest code and re-test, and also if still present please advise what browser this is.

@LucieGueuning
Copy link

LucieGueuning commented Dec 6, 2018

image
Cookies reset and cache cleared. Map zoom out still too far.

@matthewberryman
Copy link
Contributor Author

@LucieGueuning from your screenshot I can see the lngmin/lngmax contact (popup warning) aspect is resolved following your cache clear, and it seems as if the remaining aspect is just that you'd like the ability to restrict the zoom? I am just deploying a fix for that now to all environments.

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

No branches or pull requests

2 participants