uber/react-vis

Thoughts on viewBox property

Ouverte

#901 ouverte le 16 août 2018

Voir sur GitHubGuide contributeur
 (8 commentaires) (1 réaction) (0 personne assignée)JavaScript (868 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (8 615 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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

Direction de recherche
Recherchez pourquoi viewBox n'est pas pris en charge nativement dans XYPlot de react vis. Examinez l'implémentation actuelle du rendu SVG, analysez les implications potentielles sur les performances et envisagez d'autres moyens d'obtenir une mise à l'échelle réactive. Vérifiez également s'il existe des solutions de contournement ou des discussions dans la communauté.
Stack technique
javascriptreact
Domaine
frontend
Type d'issue
Recherche
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70