uber/react-vis

Thoughts on viewBox property

Offen

#901 geöffnet am 16.08.2018

Auf GitHub ansehenContributor Guide
 (8 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (868 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (8.615 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

I've been experimenting with react-vis components library and modifying it to pass a viewBox property all the way down to the svg element in XYPlot.

The results so far have been quite promising:

  • The elements inside the graph can easily be deployed in relation to each other using the internal coordinate system
  • Sizes can also be expresses in a way consistent with that system
  • The SVG scales appropriately, and it can be combined easily with flexbox.

So my question is: why is it that this doesn't have support out of the box? Is there any drawbacks I'm not noticing.

Thanks.

Contributor Guide

Research-Richtung
Untersuchen Sie, warum viewBox in react vis XYPlot nicht nativ unterstützt wird. Überprüfen Sie die aktuelle SVG Rendering Implementierung, analysieren Sie potenzielle Leistungsauswirkungen und erwägen Sie alternative Möglichkeiten zur reaktionsfähigen Skalierung. Prüfen Sie auch, ob es in der Community bereits Workarounds oder Diskussionen gibt.
Tech Stack
javascriptreact
Domain
frontend
Issue Type
Recherche
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptReact
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70