acdlite/react-remarkable

Remove dangerouslySetInnerHTML

Open

#8 geöffnet am 14. Okt. 2015

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (57 Forks)github user discovery
enhancementhelp wanted

Repository-Metriken

Stars
 (455 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Research-Richtung
Studieren Sie, wie die Bibliothek react markdown AST zum Rendern verwendet, und wenden Sie einen ähnlichen Ansatz auf react remarkable an, indem Sie es mit dem Parser von remarkable integrieren.
Tech Stack
javascriptreact
Domain
frontend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
ReactJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50