Merge pull request #53 from jduss4/removeAllMarkers

fixes onRemove so all markers are removed
dwilhelm89 · Aug 24, 2017 · cc52caa · cc52caa
2 parents 9b01d11 + 6b50440
commit cc52caa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/SliderControl.js b/SliderControl.js
@@ -83,7 +83,7 @@ L.Control.SliderControl = L.Control.extend({
 
     onRemove: function (map) {
         //Delete all markers which where added via the slider and remove the slider div
-        for (i = this.options.minValue; i < this.options.maxValue; i++) {
+        for (i = this.options.minValue; i <= this.options.maxValue; i++) {
             map.removeLayer(this.options.markers[i]);
         }
         $('#leaflet-slider').remove();

diff --git a/dist/leaflet.SliderControl.min.js b/dist/leaflet.SliderControl.min.js