Skip to content
This repository has been archived by the owner on Apr 24, 2024. It is now read-only.

Fix Overzoom issues with tile layers #1251

Closed
wants to merge 3 commits into from

Conversation

tvblomberg
Copy link

This pull request should fix issues with calculating zoom for maxNativeZoom and calculating tilesize.
Right now the maxNativeZoom option does not do anything for tileLayers. This worked correctly in version 2.1.5. Currently users will get white tiles when overzooming. This is my first pull request so please let me know if I am doing something wrong. This should fix issue #1250

Here is the example of the two versions:
2.1.5: https://jsfiddle.net/jfwkq0s4/4/
3.1.1: http://jsfiddle.net/dbpfcoqo/6/

@davidtheclark davidtheclark changed the base branch from mb-pages to publisher-production January 16, 2019 21:58
@olaf-cloudgenix
Copy link
Contributor

I hit this too when setting detectRetina to true on the tileLayer. It tries to fetch either tiles for the completely wrong geography, or tiles that don't exist (resulting in 404 errors). This pull request fixes it.

@riastrad
Copy link
Contributor

I'd like to take this along with the next minor release, but will be using #1323 to track adding it against latest master instead of wrestling with a +2 year rebase.

@riastrad riastrad closed this Mar 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants