acdlite/react-remarkable

Remove dangerouslySetInnerHTML

Aperta

#8 aperta il 14 ott 2015

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (57 fork)github user discovery
enhancementhelp wanted

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Studia come la libreria react markdown utilizza l'AST per il rendering e applica un approccio simile a react remarkable integrandolo con il parser di remarkable.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
ReactJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50