honor leaflet maxNativeZoom

[jgravois]

resolves #743

@patrickarlt was right. i just had to make sure and update the logic down in createTile() too.

[chriserickson]

Just glancing at the patch, did you want to default back to point.z with no lodmap, or default back to the zoom level as the comment states?

eg

(this._lodMap && this._lodMap[this._getZoomForUrl()]) ? 
  this._lodMap[this._getZoomForUrl()] : this._getZoomForUrl()

// instead of
(this._lodMap && this._lodMap[this._getZoomForUrl()]) ? 
  this._lodMap[this._getZoomForUrl()] : tilePoint.z

[jgravois]

good catch! with no lodmap, we should default back to the zoom level.

[chriserickson]

I manually hacked this in to our esri-leaflet-src, and it appears to work.

[jgravois]

travis will show green here as soon as #906 is merged and the build is restarted.

[ahalota]

Should this be working now? I am linking to the file shown on the main page for cdn (https://unpkg.com/[email protected]/dist/esri-leaflet-debug.js) but still not getting native zoom respected. Which version of leaflet did you have this working with?

[jgravois]

hi @ahalota! the fix was included in v2.0.7 and continues to resolve the original repro case.

1. launch https://jsfiddle.net/jagravois/z7mjvgvm/
2. whether you toggle the esri layer or the native leaflet layer, both start to resample at zoom 16.

if you have more information or can provide a demo of your own i'd be happy to take another look.

[ahalota]

Never mind. It was an odd issue in my ArcServer that was not allowing me to access the higher resolution images and turning up transparent tiles instead...