Removing map that has a tileLayer with minZoom set causes an error - undefined is not an object (evaluating 'el.style')

[bradcrawford]

• I've looked at the documentation to make sure the behavior is documented and expected
• I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
• I've searched through the issues to make sure it's not yet reported

Steps to reproduce
https://codesandbox.io/s/ywn94996wj

1. Zoom out to the point where the high res tileLayer (url set to cloudfront) disappears due to it's minZoom setting.
2. Click the Hide button.
3. Error (see image below)
4. Remove minZoom: 18
5. Repeat steps 1 & 2 above
6. No error

Expected behavior
When using a tileLayer with minZoom set and the map zoomed out beyond the value of minZoom, removing the map should not cause an error

Current behavior
When using a tileLayer with minZoom set and the map zoomed out beyond the value of minZoom, removing the map causes an error

Environment

• Leaflet version: 1.4.0
• Browser (with version): Safari 11.0.1
• OS/Platform (with version): Mac OS Sierra 10.12.6

Additional context
Uncovered while using ReactLeaflet, initial bug report here - PaulLeCam/react-leaflet#551. Further investigation revealed that it appears to be a Leaflet issue.

Minimal example reproducing the issue

https://codesandbox.io/s/ywn94996wj

• this example is as simple as possible
• this example does not rely on any third party code

[IvanSanchez]

Cannot reproduce with Firefox nor Chrome on Linux. Is this a Safari-specific issue?

[bradcrawford]

I don't believe so @IvanSanchez. I get the same error in the console in all 3 browsers. This gif is from chrome.

Here is a slightly simpler example that errors for me in both Chrome 71 and Firefox 60 as well as Safari- https://plnkr.co/edit/T4qExnzu7KzbDJkIZlJK?p=preview

[cherniavskii]

I confirm.

Looks like _destroyAnimProxy gets called before zoomend handler:

Leaflet/src/map/Map.js

Lines 1619 to 1623 in 61ef177

	this.on('load moveend', function () {
	var c = this.getCenter(),
	z = this.getZoom();
	DomUtil.setTransform(this._proxy, this.project(c, z), this.getZoomScale(z, 1));
	}, this);

[tkurki]

This is pretty bothersome, as it will kill an unsuspecting (=not expecting errors from a normal operation) React app hiding/removing a React Leaflet Map: PaulLeCam/react-leaflet#625

This is the simplest reproduction I can think of in case it is any help: https://codepen.io/tkurki/pen/XWWNBmr

[Malvoz]

This is the simplest reproduction I can think of in case it is any help: https://codepen.io/tkurki/pen/XWWNBmr

Not able to reproduce using v1.8.0: https://plnkr.co/edit/SudJnPOVbG7Y901P?preview