uber/react-vis

Thoughts on viewBox property

Open

#901 ouverte le 16 août 2018

Voir sur GitHub
 (8 commentaires) (1 réaction) (0 assignés)JavaScript (8 615 stars) (868 forks)batch import
enhancementhelp wanted

Description

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.

Guide contributeur

Stack technique
javascriptreact
Domaine
frontend
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Knowledge of ReactUnderstanding of SVG viewBox
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Review the XYPlot component in react vis to understand how SVG attributes are currently passed. Examine the viewBox property specification and how it could be integrated. Look for any prior discussion or PR references in the comments. The issue asks for support out of the box, so research the existing prop passing mechanism and propose a minimal implementation.