plotly / plotly/plotly.R

config is not overwritting `ModeBarButtons` slot

Ouverte
#2,210 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
R
Étoiles
2.7k
Forks
641
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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 ...)

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par le point d’entrée config() et reproduisez les deux exemples de l’issue en comparant plot1$x$config$modeBarButtons à plot2$x$config$modeBarButtons. Le correctif est terminé lorsqu’une configuration ultérieure de modeBarButtons met à jour la valeur précédente et que le même comportement est vérifié pour modeBarButtonsToAdd.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
r
Domaine
data-visualization
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.