Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

[CHORE]: Remove `Plotly.Snapshot.toImage`

未关闭
#8,062 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
68/100
Issue 类型
重构
描述清晰度
基本清楚
活跃度
活跃
技术栈
javascript

调研方向

Start with src/snapshot/toimage.js and the references at src/snapshot/index.js:11, devtools/test_dashboard/devtools.js:86, and src/snapshot/svgtoimg.js. Trace how the public API is exposed through src/core.js and confirm there are no remaining callers under src/. Done means the implementation and existing references are removed consistently, with the breaking-change and deprecation decision resolved for the next major release.

由索引模型根据 Issue 内容生成。

描述

chore P3 plotly-internal size: 3 task
Description

Remove Plotly.Snapshot.toImage and its implementation in src/snapshot/toimage.js.

Why should this change be made?
  • Plotly.toImage replaced it in 2016 (642ddbaee5)
  • The old function has no caller under src/ and no test coverage
Scope
  • Delete src/snapshot/toimage.js
  • Update/remove the existing references (src/snapshot/index.js:11, devtools/test_dashboard/devtools.js:86, src/snapshot/svgtoimg.js)
Notes
  • This is a breaking change, so save it for the next major release
  • This removes a public API through exports.Snapshot in src/core.js
  • We could add a deprecation notice first
主要语言
JavaScript
星标
18.3k
派生
2k
平均合并
2 天 10 小时
30 天内合并 PR
30

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

plotly/plotly.js 的其他 Issue

查看 plotly/plotly.js 的全部 Issue

相似的 Issue

更多 JavaScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。