fixes issue #86: Decorators drawn when offset larger than line length

bbecquet · Nov 12, 2018 · 742dbe2 · 742dbe2
1 parent 96858e8
commit 742dbe2
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/dist/leaflet.polylineDecorator.js b/dist/leaflet.polylineDecorator.js
@@ -82,10 +82,10 @@ function projectPatternOnPointPath(pts, pattern) {
     // 2. generate the positions of the pattern as offsets from the path start
     var positionOffsets = [];
     var positionOffset = startOffsetPixels;
-    do {
+    while (repeatIntervalPixels > 0 && positionOffset < totalPathLength - endOffsetPixels) {
         positionOffsets.push(positionOffset);
         positionOffset += repeatIntervalPixels;
-    } while (repeatIntervalPixels > 0 && positionOffset < totalPathLength - endOffsetPixels);
+    }
 
     // 3. projects offsets to segments
     var segmentIndex = 0;
@@ -182,6 +182,12 @@ function interpolateBetweenPoints(ptA, ptB, ratio) {
     });
 })();
 
+// enable rotationAngle and rotationOrigin support on L.Marker
+/**
+* Defines several classes of symbol factories,
+* to be used with L.PolylineDecorator
+*/
+
 L$1.Symbol = L$1.Symbol || {};
 
 /**

diff --git a/src/patternUtils.js b/src/patternUtils.js
@@ -66,10 +66,10 @@ function projectPatternOnPointPath(pts, pattern) {
     // 2. generate the positions of the pattern as offsets from the path start
     const positionOffsets = [];
     let positionOffset = startOffsetPixels;
-    do {
+    while(repeatIntervalPixels > 0 && positionOffset < totalPathLength - endOffsetPixels) {
         positionOffsets.push(positionOffset);
         positionOffset += repeatIntervalPixels;
-    } while(repeatIntervalPixels > 0 && positionOffset < totalPathLength - endOffsetPixels);
+    }
 
     // 3. projects offsets to segments
     let segmentIndex = 0;