Do an intersects test rather than a contains when testing for a zoom …

…in. Fixes Leaflet#344
ghybs · Apr 22, 2014 · 254198a · 254198a
1 parent f865dc1
commit 254198a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/example/old-bugs/click-cluster-at-screen-edge.html b/example/old-bugs/click-cluster-at-screen-edge.html
@@ -19,7 +19,7 @@
 <body>
 
 	<div id="map"></div>
-	<span>Bug <a href="https://github.com/danzel/Leaflet.markercluster/issues/344">#344</a>. Click the cluster at the screen edge. Map will zoom to it and its markers will appear, but it will not disappear.</span><br/>
+	<span>Bug <a href="https://github.com/leaflet/Leaflet.markercluster/issues/344">#344</a>. Click the cluster at the screen edge. Map will zoom to it and its markers will appear, but it will not disappear.</span><br/>
 	<span id="time"></span>
 	<script type="text/javascript">
 

diff --git a/src/MarkerClusterGroup.js b/src/MarkerClusterGroup.js
@@ -852,7 +852,7 @@ L.MarkerClusterGroup = L.FeatureGroup.extend({
 		//Incase we are starting to split before the animation finished
 		this._processQueue();
 
-		if (this._zoom < this._map._zoom && this._currentShownBounds.contains(this._getExpandedVisibleBounds())) { //Zoom in, split
+		if (this._zoom < this._map._zoom && this._currentShownBounds.intersects(this._getExpandedVisibleBounds())) { //Zoom in, split
 			this._animationStart();
 			//Remove clusters now off screen
 			this._topClusterLevel._recursivelyRemoveChildrenFromMap(this._currentShownBounds, this._zoom, this._getExpandedVisibleBounds());