uber/react-vis

Thoughts on viewBox property

Aberta

#901 aberto em 16 de ago. de 2018

Ver no GitHubGuia do colaborador
 (8 comentários) (1 reação) (0 responsável)JavaScript (868 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (8.615 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
Investigue por que o viewBox não é suportado nativamente no XYPlot do react vis. Revise a implementação atual de renderização SVG, analise possíveis implicações de desempenho e considere maneiras alternativas de alcançar dimensionamento responsivo. Verifique também se existem soluções alternativas ou discussões na comunidade.
Pilha de tecnologia
javascriptreact
Domain
frontend
Tipo Issue
Pesquisa
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptReact
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70