Commit 1e2e1c83ea93c3c019daed29e217d37e84dad882 - leaflet

fix(Map): rm moveend listener from setMaxBounds (#6958) Calling `map.setMapBounds`, adds the `"moveend"` event listener `_panInsideMaxBounds`. Previously, this did not get cleaned up via `remove`. Simon Legner authored 4 years ago Vladimir Agafonkin committed 4 years ago

2 changed file(s) with 2 addition(s) and 1 deletion(s). Raw diff Collapse all Expand all

-1

spec/suites/map/MapSpec.js less more

71	71
72	72	it("does not throw if removed during animation", function () {
73	73	var container = document.createElement('div'),
74		map = new L.Map(container).setView([0, 0], 1);
	74	map = new L.Map(container).setView([0, 0], 1).setMaxBounds([[0, 1], [2, 3]]);
75	75
76	76	// Force creation of animation proxy,
77	77	// otherwise browser checks disable it

-0

src/map/Map.js less more

751	751	remove: function () {
752	752
753	753	this._initEvents(true);
	754	this.off('moveend', this._panInsideMaxBounds);
754	755
755	756	if (this._containerId !== this._container._leaflet_id) {
756	757	throw new Error('Map container is being reused by another instance');