-
Notifications
You must be signed in to change notification settings - Fork 675
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
Allow user to use maxNativeZoom and maxZoom to allow pixelized tile (instead of a grey tile) #323
base: master
Are you sure you want to change the base?
Conversation
@jieter any thoughts about it ? |
I'm not sure about this. I understand the use case to override maxzoom/maxnativezoom, but I don't like to do it by default. |
@JoffreyBourdieux may implement this as an option ? |
By default you cannot use the maxNativeZoom property of Leaflet, so you can't zoom "above" the max zoom of the layer (a gray layer is returned instead). This update seem to not change the original behavior of the plugin, since if you dont tell Leaflet to use the property "maxNativeZoom" it will simply stop the maximum zoom to the one provided by "maxZoom"
I'll look into it |
Hi, It is now implemented as an option.
It use a key "useNativeZoom" to detect if the user want to override the default maxZoom value, then clone the maxZoom value inside a maxNativeZoom key and override the maxZoom from the providers list value with the user value. Tell me if this modification suits you. |
Hi, I'm guessing this hasn't been implemented yet? |
Hi,
A small update I have done for a site i'm working on, it allow a layer to be pixelized above maxZoom and not showing a grey tile (like originaly).
I have done this because when you use a provider it "lock" the maxZoom value to the one set in the providers list.
But if you want to use maxNativeZoom, the plugin will still try to fetch tiles with greater maxZoom than the one allowed and it will return grey tiles.
Thanks in advance :)