Add stroke and fill arguments to setStyleFast

Should pass pathOptions but can't be bothered to fix it rn

Also fix devtools::document() unexporting setStyleFast.

Author: cmcaine

R/methods.R

@@ -52,8 +52,9 @@ setStyle = function(map, group, styles, label = NULL, offset = 0) {
     invokeMethod(map, NULL, "setStyle", group, styles, label, offset)
 }
 
-setStyleFast = function(map, group, color = NULL, weight = NULL, label = NULL) {
-    invokeMethod(map, NULL, "setStyleFast", group, color, weight, label)
+#' @export
+setStyleFast = function(map, group, color = NULL, weight = NULL, label = NULL, stroke = NULL, fill = NULL) {
+    invokeMethod(map, NULL, "setStyleFast", group, color, weight, label, stroke, fill)
 }
 
 #' @describeIn map-methods Flys to a given location/zoom-level using smooth pan-zoom.

inst/htmlwidgets/leaflet.js

@@ -1303,7 +1303,7 @@ methods.setStyle = function (group, styles, labels) {
 };
 
 /** Much more performant way to style loaded geometry */
-methods.setStyleFast = function (group, colors, weights, labels) {
+methods.setStyleFast = function (group, colors, weights, labels, strokes, fills) {
   window.map = this;
   var layers = this.layerManager.getLayerGroup(group).getLayers();
 
@@ -1324,6 +1324,18 @@ methods.setStyleFast = function (group, colors, weights, labels) {
       layers[_i3].setStyle({ weight: weights[_i3] });
     }
   }
+
+  if (strokes) {
+    for (var _i4 = 0; _i4 < strokes.length; _i4++) {
+      layers[_i4].setStyle({ stroke: strokes[_i4] });
+    }
+  }
+
+  if (fills) {
+    for (var _i5 = 0; _i5 < fills.length; _i5++) {
+      layers[_i5].setStyle({ fill: fills[_i5] });
+    }
+  }
 };
 
 function mouseHandler(mapId, layerId, group, eventName, extraInfo) {

javascript/src/methods.js

@@ -31,7 +31,7 @@ methods.setStyle = function(group, styles, labels, offset = 0) {
 };
 
 /** Much more performant way to style loaded geometry */
-methods.setStyleFast = function(group, colors, weights, labels) {
+methods.setStyleFast = function(group, colors, weights, labels, strokes, fills) {
   window.map = this;
   let layers = this.layerManager.getLayerGroup(group).getLayers();
 
@@ -43,13 +43,25 @@ methods.setStyleFast = function(group, colors, weights, labels) {
 
   if (colors) {
     for (let i = 0; i < colors.length; i++) {
-      layers[i].setStyle({color: colors[i], fillColor: colors[i]})
+      layers[i].setStyle({color: colors[i], fillColor: colors[i]});
     }
   }
 
   if (weights) {
     for (let i = 0; i < weights.length; i++) {
-      layers[i].setStyle({weight: weights[i]})
+      layers[i].setStyle({weight: weights[i]});
+    }
+  }
+
+  if (strokes) {
+    for (let i = 0; i < strokes.length; i++) {
+      layers[i].setStyle({stroke: strokes[i]});
+    }
+  }
+
+  if (fills) {
+    for (let i = 0; i < fills.length; i++) {
+      layers[i].setStyle({fill: fills[i]});
     }
   }
 };