acdlite / acdlite/react-remarkable

Remove dangerouslySetInnerHTML

Aperta
#8 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
enhancement help wanted
Lingua principale
JavaScript
Stelle
457
Fork
57
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

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 per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.