mermaidWithDarkTheme slaat dark-directive over bij eigen %%{init}-directive

brenno commented

2026-09-02 13:26:03 +00:00

Owner

Probleem

mermaidWithDarkTheme in lib/widgets/reader/doc_mermaid_view.dart slaat zijn dark-directive over als de bron al met %%{ begint:

String mermaidWithDarkTheme(String source) {
  final head = source.trimLeft();
  if (head.startsWith('%%{') || head.startsWith('---')) return source;
  return '%%{init: {"theme":"dark"}}%%\n$source';
}

De bedoeling is terecht: een diagram met een eigen %%{init}-directive moet die behouden. Maar het neveneffect is dat een diagram met een eigen init-directive (bijv. %%{init: {"themeVariables": {"primaryColor": "#ff0"}}}%%) in donker thema licht blijft, omdat de theme: dark-directive nooit wordt toegevoegd.

Oplossingsrichting

Samenvoegen in plaats van overslaan: als de bron al een %%{init: ...}%%-directive heeft, de theme: dark daarin injecteren (bestaande keys behouden, theme toevoegen of overschrijven). Als de bron met --- (YAML frontmatter) begint, de directive erna plaatsen.

Beeldkeuring-controlepunt (geen bevinding)

Daarnaast verdient het sequence-diagram een beeldkeuring. In een nabootsing kwam een Note als donker vlak met verschoven tekst uit de inliner, en verdwenen de autonumber-cirkels. Dit is niet in flutter_svg zelf geverifieerd, dus dit is een controlepunt, geen bevinding — bij het fixen van bovenstaand punt meenemen in de visuele controle.

Referenties

lib/widgets/reader/doc_mermaid_view.dart regel 207-211: mermaidWithDarkTheme
test/document_reader_test.dart regel 435-450: bestaande tests voor mermaidWithDarkTheme

## Probleem `mermaidWithDarkTheme` in `lib/widgets/reader/doc_mermaid_view.dart` slaat zijn dark-directive over als de bron al met `%%{` begint: ```dart String mermaidWithDarkTheme(String source) { final head = source.trimLeft(); if (head.startsWith('%%{') || head.startsWith('---')) return source; return '%%{init: {"theme":"dark"}}%%\n$source'; } ``` De bedoeling is terecht: een diagram met een eigen `%%{init}`-directive moet die behouden. Maar het neveneffect is dat een diagram met een eigen init-directive (bijv. `%%{init: {"themeVariables": {"primaryColor": "#ff0"}}}%%`) in donker thema **licht** blijft, omdat de `theme: dark`-directive nooit wordt toegevoegd. ## Oplossingsrichting Samenvoegen in plaats van overslaan: als de bron al een `%%{init: ...}%%`-directive heeft, de `theme: dark` daarin injecteren (bestaande keys behouden, `theme` toevoegen of overschrijven). Als de bron met `---` (YAML frontmatter) begint, de directive erna plaatsen. ## Beeldkeuring-controlepunt (geen bevinding) Daarnaast verdient het sequence-diagram een beeldkeuring. In een nabootsing kwam een Note als donker vlak met verschoven tekst uit de inliner, en verdwenen de autonumber-cirkels. Dit is niet in `flutter_svg` zelf geverifieerd, dus dit is een **controlepunt**, geen bevinding — bij het fixen van bovenstaand punt meenemen in de visuele controle. ## Referenties - `lib/widgets/reader/doc_mermaid_view.dart` regel 207-211: `mermaidWithDarkTheme` - `test/document_reader_test.dart` regel 435-450: bestaande tests voor `mermaidWithDarkTheme`

brenno added the

in-progress

label

2026-09-02 13:31:57 +00:00

brenno referenced this issue from a commit

2026-09-02 14:08:10 +00:00

fix(mermaid): dark-theme merge en schalen in Pagina's-weergave (#1921, #1922)

brenno referenced this issue

2026-09-02 14:09:16 +00:00

fix(mermaid): dark-theme merge en schalen in Pagina's-weergave (#1921, #1922) #1924

brenno commented

2026-09-02 15:14:17 +00:00

Author

Owner

Opgepakt (sessie hervat na een crash). Tak: fix/mermaid-dark-theme-and-scaling, PR #1924. Reikwijdte: mermaidWithDarkTheme in lib/widgets/reader/doc_mermaid_view.dart.

brenno referenced this issue from a commit

2026-09-02 15:42:58 +00:00

docs(changelog): de mermaid-reparaties in het Unreleased-logboek (#1921, #1922)

brenno commented

2026-09-02 16:04:58 +00:00

Author

Owner

Het beeldkeuring-controlepunt (sequence-diagram: Note-vlak en autonumber-cirkels) is NIET visueel getoetst. De schermtoegang voor computer-use werd geweigerd, en dat is de enige route om OciDeck echt te bekijken. De code-reparatie zelf is wel gedekt: mermaidWithDarkTheme heeft drie tests op de samenvoeging (bestaande init-directive, behoud van andere sleutels, plaatsing na YAML-frontmatter) en de volledige make check staat groen.

Het controlepunt blijft dus open staan. Wie het alsnog wil doen heeft twee wegen: de app draaien met toegestane schermbediening, of een integratietest op macOS die de RepaintBoundary naar PNG schrijft — daar is de WebView wél beschikbaar, anders dan onder flutter test.

Het beeldkeuring-controlepunt (sequence-diagram: Note-vlak en autonumber-cirkels) is NIET visueel getoetst. De schermtoegang voor computer-use werd geweigerd, en dat is de enige route om OciDeck echt te bekijken. De code-reparatie zelf is wel gedekt: mermaidWithDarkTheme heeft drie tests op de samenvoeging (bestaande init-directive, behoud van andere sleutels, plaatsing na YAML-frontmatter) en de volledige make check staat groen. Het controlepunt blijft dus open staan. Wie het alsnog wil doen heeft twee wegen: de app draaien met toegestane schermbediening, of een integratietest op macOS die de RepaintBoundary naar PNG schrijft — daar is de WebView wél beschikbaar, anders dan onder flutter test.

brenno referenced this issue from a commit

2026-09-02 16:41:45 +00:00

Merge pull request 'fix(mermaid): dark-theme merge en schalen in Pagina's-weergave (#1921, #1922)' (#1924) from fix/mermaid-dark-theme-and-scaling into main

brenno commented

2026-09-02 16:42:14 +00:00

Author

Owner

Opgelost en op main geverifieerd: merge-commit 5f113a229 (PR #1924), reparatie in a9731614c. mermaidWithDarkTheme voegt theme:dark nu samen in een bestaande %%{init}-directive in plaats van hem over te slaan, met behoud van andere sleutels, en zet de directive ná YAML-frontmatter. Drie tests dekken die drie gevallen.

Wat er NIET in zit: het beeldkeuring-controlepunt op het sequence-diagram. Zie de reactie hierboven — de schermtoegang werd geweigerd, dus dat blijft ongetoetst. Dat was in de issuetekst zelf al als controlepunt en niet als bevinding aangemerkt; ik sluit hier op de bevinding, niet op het controlepunt.

Opgelost en op main geverifieerd: merge-commit 5f113a229 (PR #1924), reparatie in a9731614c. mermaidWithDarkTheme voegt theme:dark nu samen in een bestaande %%{init}-directive in plaats van hem over te slaan, met behoud van andere sleutels, en zet de directive ná YAML-frontmatter. Drie tests dekken die drie gevallen. Wat er NIET in zit: het beeldkeuring-controlepunt op het sequence-diagram. Zie de reactie hierboven — de schermtoegang werd geweigerd, dus dat blijft ongetoetst. Dat was in de issuetekst zelf al als controlepunt en niet als bevinding aangemerkt; ik sluit hier op de bevinding, niet op het controlepunt.

brenno closed this issue

2026-09-02 16:42:16 +00:00

Rows
Columns

mermaidWithDarkTheme slaat dark-directive over bij eigen %%{init}-directive #1922

Probleem

Oplossingsrichting

Beeldkeuring-controlepunt (geen bevinding)

Referenties