plotly / plotly/plotly.R

config is not overwritting `ModeBarButtons` slot

Abierto
#2,210 1 comentario 0 reacciones 0 asignados Ver en GitHub

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

I have noticed that after placing a ModeBarButtons configuration, it is not overwritten / updated downstream.

Reproducible example:

library(plotly)
#> Loading required package: ggplot2
#> 
#> Attaching package: 'plotly'
#> The following object is masked from 'package:ggplot2':
#> 
#>     last_plot
#> The following object is masked from 'package:stats':
#> 
#>     filter
#> The following object is masked from 'package:graphics':
#> 
#>     layout
  # This plotly only has the `toImage` modeBarButton
  plot1 <- plot_ly() %>%
    config(displaylogo = FALSE) %>%
    config(modeBarButtons = list(list("toImage")))
  plot1$x$config$modeBarButtons
#> [[1]]
#> [[1]][[1]]
#> [1] "toImage"
  
  # This plotly also has the `toImage` modeBarButton only
  plot2 <- plot_ly() %>%
    config(displaylogo = FALSE) %>%
    config(modeBarButtons = list(list("toImage"))) %>% 
    config(modeBarButtons = list(list("zoom")))
  plot2$x$config$modeBarButtons
#> [[1]]
#> [[1]][[1]]
#> [1] "toImage"

Created on 2022-11-29 with reprex v2.0.2

This holds true for config(modeBarButtonsToAdd ...)

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza con el punto de entrada config() y reproduce los dos ejemplos del issue, comparando plot1$x$config$modeBarButtons con plot2$x$config$modeBarButtons. La corrección estará completa cuando una configuración posterior de modeBarButtons actualice el valor anterior y se verifique el mismo comportamiento para modeBarButtonsToAdd.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
r
Área
data-visualization
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.