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

Offen

#8,062 erstellt am 21.9.2026 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 68/100

Issue-Typ: Refactoring
Klarheit: Größtenteils klar
Aktivitätsstatus: Aktiv
Tech-Stack: javascript
Bereich: data-visualization

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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

Vorherrschende Sprache: JavaScript
Sterne: 18.3k
Forks: 2k
Ø Merge: 2 T. 10 Std.
Gemergte PRs (30 T.): 30

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus plotly/plotly.js

[CHORE]: Warn on config values that do not match the schema Offen

chore P3 plotly-internal size: 3 task

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 77/100

plotly/plotly.js#8064 · Erstellt am 21.9.2026
[CHORE]: Change default to `true` for `responsive` config attribute Offen

chore P1 plotly-internal size: 1 task

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 82/100

plotly/plotly.js#8034 · Erstellt am 10.9.2026
[CHORE]: Deprecate plotly.js-mapbox* npm packages Offen

chore P3 plotly-internal size: 1 task

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100

plotly/plotly.js#8029 · Erstellt am 08.9.2026
[mkdocs] Update link in Edit this Page button Offen

bug

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100

plotly/plotly.js#7648 · 3 Kommentare · Erstellt am 18.11.2025
BUILDING.md: minor typo and some Markdown syntax errors Offen

bug infrastructure P2

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 65/100

plotly/plotly.js#6930 · Erstellt am 15.3.2024

Alle Issues in plotly/plotly.js