Update L.Polyline's _pxBounds when their stroke weight changes (#6516)

Leaflet · Feb 20, 2019 · 13f649d · 13f649d
1 parent dc6a0ae
commit 13f649d
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 6 deletions.
diff --git a/src/layer/vector/Path.js b/src/layer/vector/Path.js
@@ -105,6 +105,9 @@ export var Path = Layer.extend({
 		Util.setOptions(this, style);
 		if (this._renderer) {
 			this._renderer._updateStyle(this);
+			if (this.options.stroke && style.hasOwnProperty('weight')) {
+				this._updateBounds();
+			}
 		}
 		return this;
 	},

diff --git a/src/layer/vector/Polyline.js b/src/layer/vector/Polyline.js
@@ -203,16 +203,21 @@ export var Polyline = Path.extend({
 		this._rings = [];
 		this._projectLatlngs(this._latlngs, this._rings, pxBounds);
 
-		var w = this._clickTolerance(),
-		    p = new Point(w, w);
-
 		if (this._bounds.isValid() && pxBounds.isValid()) {
-			pxBounds.min._subtract(p);
-			pxBounds.max._add(p);
-			this._pxBounds = pxBounds;
+			this._rawPxBounds = pxBounds;
+			this._updateBounds();
 		}
 	},
 
+	_updateBounds: function () {
+		var w = this._clickTolerance(),
+		    p = new Point(w, w);
+		this._pxBounds = new Bounds([
+			this._rawPxBounds.min.subtract(p),
+			this._rawPxBounds.max.add(p)
+		]);
+	},
+
 	// recursively turns latlngs into a set of rings with projected coordinates
 	_projectLatlngs: function (latlngs, result, projectedBounds) {
 		var flat = latlngs[0] instanceof LatLng,