don't debounce by default

Author: cpsievert

R/highlight.R

@@ -43,8 +43,9 @@
 #' @param opacityDim a number between 0 and 1 used to reduce the
 #' opacity of non-selected traces (by multiplying with the existing opacity).
 #' @param selected attributes of the selection, see [attrs_selected()].
-#' @param debounce amount of time to wait before firing an event (in milliseconds). 
-#' This is especially useful when `on = "plotly_hover"` to avoid firing too many events
+#' @param debounce amount of time to wait before firing an event (in milliseconds).
+#' The default of 0 means do not debounce at all. 
+#' Debouncing is mainly useful when `on = "plotly_hover"` to avoid firing too many events
 #' when users clickly move the mouse over relevant graphical marks.
 #' @param ... currently not supported.
 #' @export
@@ -83,7 +84,7 @@ highlight <- function(p, on = "plotly_click", off,
                       dynamic = FALSE, color = NULL,
                       selectize = FALSE, defaultValues = NULL,
                       opacityDim = getOption("opacityDim", 0.2), 
-                      selected = attrs_selected(), debounce = 1,
+                      selected = attrs_selected(), debounce = 0,
                       ...) {
 
   # currently ... is not-supported and will catch 

inst/htmlwidgets/plotly.js

@@ -471,8 +471,10 @@ HTMLWidgets.widget({
           }
         }
       };
-      
-      graphDiv.on(x.highlight.on, debounce(turnOn, x.highlight.debounce));
+      if (x.highlight.debounce > 0) {
+        turnOn = debounce(turnOn, x.highlight.debounce);
+      }
+      graphDiv.on(x.highlight.on, turnOn);
 
       graphDiv.on(x.highlight.off, function turnOff(e) {
         // remove any visual clues