fix(codec): blokinhoud in een lijstitem reist verliesvrij door de visuele modus (#1925) #1936

Closed

brenno wants to merge 2 commits from fix/codec-list-item-block-content-v2 into main

brenno commented

2026-09-03 05:57:50 +00:00

Owner

Samenvatting

Quills documentmodel is vlak: list en codeBlock zijn exclusieve blokattributen, dus een codeblok in een lijstitem kon niet als kind van de bulletregel reizen. MarkdownToDelta plakte de inhoud aan de lijstitem-tekst met een x-md-codeblock-lang-attribuut, waardoor de fence en de inspringing stil verloren gingen bij de rondgang. Hetzelfde gold voor tabellen en mermaid-diagrammen in lijstitems.

De codec voert blokinhoud in een lijstitem nu als x-embed-list-block-embed met de inspringing bewaard:

Heenweg (MarkdownToDelta): onderschept pre en embeds in visitElementBefore wanneer _listItemIndent >= 0, sluit de lijstitem-regel af, reconstrueert de ruwe markdown (fence voor codeblokken, data-attribuut voor embeds), en draagt het blok als embed met de juiste inspringing (2 spaties per niveau voor ongeordende lijsten, 3 voor geordende).
Terugweg (DeltaToMarkdown): de EmbeddableListBlock.toMdSyntax handler schrijft de ruwe markdown verbatim terug.
Lijstnummering: de afsluitende regel krijgt een indent-attribuut zodat _prefixNumber de nummering niet verbreekt — de teller slaat ingesprongen niet-lijst-regels over.

Wat er verandert aan het bestandsformaat

Niets. De round-trip was al supposed to be lossless; deze fix maakt het daadwerkelijk lossless. De Markdown die er uit komt is standaard GFM met ingesprongen blokinhoud in lijstitems — precies wat de CommonMark/GFM-spec voorschrijft voor geneste blokken.

Wat de bewaker zou moeten weten

Deze wijziging raakt de vendored markdown_quill fork (third_party/) en de codec die de Markdown-round-trip verzorgt. De verandering is een bugfix, geen format- of opslagwijziging: de visuele modus herschreef blokinhoud in lijstitems stil, en dit herstelt de oorspronkelijke Markdown. Geen nieuwe afhankelijkheid, geen uitgaand verkeer, geen publieke belofte die verandert.

Testplan

Regressietests voor codeblok, tabel, mermaid en geordende lijst in test/markdown_quill_codec_test.dart
Bestaande codec-tests, round-trip tests, visual-save tests, pentest tests en source-map tests groen (211 tests)
Statische poorten groen (format, analyze, conventions, dead-code, comment-language, hardcoded-text)
make check volledig groen — 10 pre-existing test-failures op origin/main (logo, l10n, SBOM, docs) zijn niet door deze wijziging veroorzaakt

Generated with Devin

## Samenvatting Quills documentmodel is vlak: `list` en `codeBlock` zijn exclusieve blokattributen, dus een codeblok in een lijstitem kon niet als kind van de bulletregel reizen. `MarkdownToDelta` plakte de inhoud aan de lijstitem-tekst met een `x-md-codeblock-lang`-attribuut, waardoor de fence en de inspringing stil verloren gingen bij de rondgang. Hetzelfde gold voor tabellen en mermaid-diagrammen in lijstitems. De codec voert blokinhoud in een lijstitem nu als `x-embed-list-block`-embed met de inspringing bewaard: - **Heenweg** (`MarkdownToDelta`): onderschept `pre` en embeds in `visitElementBefore` wanneer `_listItemIndent >= 0`, sluit de lijstitem-regel af, reconstrueert de ruwe markdown (fence voor codeblokken, `data`-attribuut voor embeds), en draagt het blok als embed met de juiste inspringing (2 spaties per niveau voor ongeordende lijsten, 3 voor geordende). - **Terugweg** (`DeltaToMarkdown`): de `EmbeddableListBlock.toMdSyntax` handler schrijft de ruwe markdown verbatim terug. - **Lijstnummering**: de afsluitende regel krijgt een `indent`-attribuut zodat `_prefixNumber` de nummering niet verbreekt — de teller slaat ingesprongen niet-lijst-regels over. ## Wat er verandert aan het bestandsformaat Niets. De round-trip was al supposed to be lossless; deze fix maakt het daadwerkelijk lossless. De Markdown die er uit komt is standaard GFM met ingesprongen blokinhoud in lijstitems — precies wat de CommonMark/GFM-spec voorschrijft voor geneste blokken. ## Wat de bewaker zou moeten weten Deze wijziging raakt de vendored `markdown_quill` fork (third_party/) en de codec die de Markdown-round-trip verzorgt. De verandering is een bugfix, geen format- of opslagwijziging: de visuele modus herschreef blokinhoud in lijstitems stil, en dit herstelt de oorspronkelijke Markdown. Geen nieuwe afhankelijkheid, geen uitgaand verkeer, geen publieke belofte die verandert. ## Testplan - [x] Regressietests voor codeblok, tabel, mermaid en geordende lijst in `test/markdown_quill_codec_test.dart` - [x] Bestaande codec-tests, round-trip tests, visual-save tests, pentest tests en source-map tests groen (211 tests) - [x] Statische poorten groen (format, analyze, conventions, dead-code, comment-language, hardcoded-text) - [ ] `make check` volledig groen — 10 pre-existing test-failures op `origin/main` (logo, l10n, SBOM, docs) zijn niet door deze wijziging veroorzaakt Generated with [Devin](https://devin.ai)

brenno added 2 commits

2026-09-03 05:57:51 +00:00

fix(cockpit): tekst van het glas af — uitleesvenster in de flank, label eronder (#1929 ) 3f312523c8

fix(codec): blokinhoud in een lijstitem reist verliesvrij door de visuele modus (#1925 )

scans / scans (pull_request) Successful in 2m8s

Details

static-gate / static-gate (pull_request) Failing after 5m35s

Details

af4ec625c9

Quills documentmodel is vlak: `list` en `codeBlock` zijn exclusieve
blokattributen, dus een codeblok in een lijstitem kon niet als kind van
de bulletregel reizen. `MarkdownToDelta` plakte de inhoud aan de
lijstitem-tekst met een `x-md-codeblock-lang`-attribuut, waardoor de
fence en de inspringing stil verloren gingen bij de rondgang. Hetzelfde
gold voor tabellen en mermaid-diagrammen in lijstitems.

De codec voert blokinhoud in een lijstitem nu als `x-embed-list-block`-
embed met de inspringing bewaard. De heenweg onderschept `pre` en embeds
in `visitElementBefore` wanneer `_listItemIndent >= 0`, sluit de
lijstitem-regel af, en draagt het blok als embed met de juiste
inspringing (2 spaties per niveau voor ongeordende lijsten, 3 voor
geordende). De terugweg schrijft de ruwe markdown verbatim terug. De
afsluitende regel krijgt een `indent`-attribuut zodat de lijstnummering
niet verbreekt (`_prefixNumber` slaat ingesprongen niet-lijst-regels
over).

Regressietests voor codeblok, tabel, mermaid en geordende lijst.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

brenno closed this pull request

2026-09-03 08:00:36 +00:00