Delegate node walking to others
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
- Tipo di issue
- Refactoring
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- tooling
Direzione di ricerca
Inizia esaminando l’interfaccia attuale di walker e il modo in cui l’AST parser produce l’output, quindi confronta gli approcci browser Document e JSDOM descritti nell’issue. Il lavoro sarà completato quando il parser costruirà direttamente un DOM, verrà rimosso l’ambito relativo a walker e i consumer potranno sanificare, interrogare, aggiungere o renderizzare il document risultante tramite le DOM APIs esistenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
To reduce the scope of this package (easier to maintain), how about this:
- Remove the
walkerinterface. Instead, the AST parser builds into a real DOM directly.- On the browser, this would be an instance of
Document(that's what e.g. DOM Parser would output). - In node, something like JSDOM can be used.
- So either of those two would be a dependency for this library.
- On the browser, this would be an instance of
- That way, any sanitization or other DOM modification could either use an existing library, such as DOMPurify or write their own code using e.g.
myDocument.querySelectorAll. - Rendering to an HTML string be as simple as
myDocument.innerHTML, and others will likely just append that doc into the DOM directly, without ever producing an HTML string (perf win).
This change would radically lower the scope of this package (building on the shoulder of giants; existing web APIs basically), and would still preserve everything that's unique about this library, which is markdown parsing.
- Lingua principale
- JavaScript
- Stelle
- 1.6k
- Fork
- 231
- Merge medio
- 2g 17h
- PR unite (30g)
- 2
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di commonmark/commonmark.js
-
White space handling looks inconsistent between HTML block element vs HTML inline elementForse già presa @mamicicekel l’ha presa 3 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
commonmark/commonmark.js#303 · 2 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
commonmark/commonmark.js#298 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
commonmark/commonmark.js#287 · 1 reazione ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 20/100
commonmark/commonmark.js#275 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
commonmark/commonmark.js#264 · 1 reazione ·
Tutte le issue di commonmark/commonmark.js
Issue simili
-
[BUG] Multi-day events show "Ended" while still in progressForse già presa @tarunagnihotri534 l’ha presa oggi. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
data-umbrella/du-event-board#231 · 2 commenti ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
NaturalIntelligence/fast-xml-parser#888 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 80/100
USACE/chart-docs#766 ·
-
bug callouts regression revealjs
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
quarto-dev/quarto-cli#15014 ·
I maintainer di solito rispondono entro 1 giorno