plotly / plotly/plotly.js

Use viewbox for svg elements

Offen
#7,064 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

feature P3
Vorherrschende Sprache
JavaScript
Sterne
18.3k
Forks
2k
Ø Merge
2 T. 12 Std.
Gemergte PRs (30 T.)
28

Beschreibung

Currently plotly uses a fixed width/height layout for the svg elements. Even when using a responsive layout, the svg gets sized to fixed width/height box according to the outer div. This means that any css styling, and in particular @media print do not get applied. Is there any reason not to use a layout that is actually use controllable / customizable by setting viewbox parameters instead of fixed pixel sizes for the plots containers?

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Das Issue nennt keine Dateien, Tests oder Einstiegspunkte; beginne damit nachzuverfolgen, wie plotly.js die Größe von SVG-Elementen festlegt und responsive Layouts handhabt. Vergleiche das aktuelle Verhalten mit fester Breite/Höhe mit der angeforderten viewBox-basierten Anpassung und definiere „fertig“ so, dass CSS einschließlich @media print die Plotgröße steuern kann, ohne responsive Plots zu beeinträchtigen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
data-visualization, frontend
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.