acdlite / acdlite/react-remarkable

Remove dangerouslySetInnerHTML

Open
#8 1 comment 0 reactions 0 assignees View on GitHub
enhancement help wanted
Dominant language
JavaScript
Stars
457
Forks
57
PR merge metrics
No merged PRs in 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.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.