config is not overwritting `ModeBarButtons` slot
未关闭
还没有人认领这个 Issue。
- 主要语言
- R
- 星标
- 2.7k
- 派生
- 641
- PR 合并指标
- 30 天内没有已合并 PR
描述
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 ...)
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
从 config() 入口开始,重现 issue 中的两个示例,比较 plot1$x$config$modeBarButtons 与 plot2$x$config$modeBarButtons。当后续的 modeBarButtons 配置能够更新之前的值,并且对 modeBarButtonsToAdd 验证出相同的行为时,修复就完成了。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- r
- 领域
- data-visualization
- Issue 类型
- 缺陷
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 45/100