uber/react-vis

Thoughts on viewBox property

Aperta

#901 aperta il 16 ago 2018

Vedi su GitHubGuida contributor
 (8 commenti) (1 reazione) (0 assegnatari)JavaScript (868 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (8615 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga perché viewBox non è supportato nativamente in react vis XYPlot. Esamina l'implementazione corrente del rendering SVG, analizza le potenziali implicazioni sulle prestazioni e considera modi alternativi per ottenere un ridimensionamento reattivo. Controlla anche se esistono soluzioni alternative o discussioni nella community.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Ricerca
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70