config modeBarButtonsToRemove fails if length is 1
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- R
- Estrellas
- 2.7k
- Forks
- 641
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
The plot silently fails to render and throws a javascript error when listing only 1 button for removal with config(modeBarButtonsToRemove = c(...))
# Basic failure
plot_ly(data = mtcars, x = ~mpg, y = ~wt) %>%
config(modeBarButtonsToRemove = c("hoverCompareCartesian"))
plot_ly(data = mtcars, x = ~mpg, y = ~wt) %>%
config(modeBarButtonsToRemove = c("zoom2d"))
# Fails even when button is superfluous ("zoom3d" not needed for a 2d plot)
plot_ly(data = mtcars, x = ~mpg, y = ~wt) %>%
config(modeBarButtonsToRemove = c("zoom3d"))
# Fails when passed as a scalar
plot_ly(data = mtcars, x = ~mpg, y = ~wt) %>%
config(modeBarButtonsToRemove = "hoverCompareCartesian")
// javscript error from browser
Uncaught TypeError: x.config.modeBarButtonsToRemove.push is not a function
at Object.renderValue (plotly.js:155)
at exports.OutputBinding.shinyBinding.renderValue (htmlwidgets.js:516)
at exports.OutputBinding.onValueChange (output_binding.js:16)
at exports.OutputBinding.delegator.(:7541/anonymous function) [as onValueChange] (http://127.0.0.1:7541/htmlwidgets-1.3/htmlwidgets.js:112:23)
at OutputBindingAdapter.onValueChange (output_binding_adapter.js:21)
at ShinyApp.receiveOutput (shinyapp.js:332)
at ShinyApp.<anonymous> (shinyapp.js:544)
at ShinyApp._sendMessagesToHandlers (shinyapp.js:529)
at ShinyApp.dispatchMessage (shinyapp.js:515)
at WebSocket.c.onmessage (shinyapp.js:112)
renderValue @ plotly.js:155
shinyBinding.renderValue @ htmlwidgets.js:516
onValueChange @ output_binding.js:16
delegator.(anonymous function) @ htmlwidgets.js:112
onValueChange @ output_binding_adapter.js:21
receiveOutput @ shinyapp.js:332
(anonymous) @ shinyapp.js:544
_sendMessagesToHandlers @ shinyapp.js:529
dispatchMessage @ shinyapp.js:515
c.onmessage @ shinyapp.js:112
Adding a second, even irrelevant value, fixes the error.
# Remove 2 buttons
plot_ly(data = mtcars, x = ~mpg, y = ~wt) %>%
config(modeBarButtonsToRemove = c("hoverCompareCartesian","zoom2d"))
# zoom3d would not appear by default because this is a 2d plot, but adding it to the argument prevents the error
plot_ly(data = mtcars, x = ~mpg, y = ~wt) %>%
config(modeBarButtonsToRemove = c("hoverCompareCartesian","zoom3d"))
I am using plotly 4.8.0
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Reproduce el fallo con los ejemplos de R utilizando valores de modeBarButtonsToRemove de un elemento y escalares. Comienza en el punto de entrada renderValue mostrado en la traza de la pila del navegador y, a continuación, verifica que las configuraciones con un solo botón se rendericen sin errores, mientras que el comportamiento existente con varios botones permanece intacto.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, r
- Área
- data-visualization, frontend
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 38/100