acdlite/react-remarkable

Remove dangerouslySetInnerHTML

Aberta

#8 aberto em 14 de out. de 2015

Ver no GitHub
 (1 comentário) (0 reação) (0 responsável)JavaScript (57 forks)github user discovery
enhancementhelp wanted

Métricas do repositório

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

Description

Was quite enthused to see this but turned out to be a wrapper. Is there any possibility of adding true React support via AST, along the lines of https://github.com/rexxars/react-markdown ?

That is the "correct" approach. React does DOM diffs, so with ASTs it will only render the changed parts instead of re-rendering everything like right now this repo does, which brings perf and other improvements.

Guia do colaborador

Direção de pesquisa
Estude como a biblioteca react markdown usa AST para renderizar e aplique abordagem semelhante ao react remarkable, integrando com o parser do remarkable.
Pilha de tecnologia
javascriptreact
Domain
frontend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
ReactJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50