refactor(documentmodus): de drie weergavestanden in een eigen part #1515

Merged

brenno merged 1 commit from refactor/documentscherm-standen into main

2026-08-17 10:18:43 +00:00

brenno commented

2026-08-17 10:12:30 +00:00

Owner

Sluit #1509.

Waarom

document_editor_screen.dart stuitte tijdens #1505 en #1506 twee keer op zijn regelplafond, en beide keren kostte een toevoeging van drie regels eerst een verhuizing. Dat is een teken dat het scherm te veel doet, niet dat het plafond te laag staat.

Wat

De drie weergavestanden — bron naast weergave, het document op pagina's, en het visuele schrijfvlak — plus hun toebehoren (_wysiwygEditor, _editor, _preview) staan nu in lib/widgets/parts/document_editor_layouts.dart, als extension _DocumentEditorLayouts on _DocumentEditorScreenState. Zelfde library, zelfde staat, zelfde gedrag; het scherm houdt de staat en de bedrading over.

Van 1168 naar 953 regels. De basislijn in tool/check_conventions.dart zakt mee van 1237 naar 960 — een plafond dat driehonderd regels boven de werkelijkheid hangt bewaakt niets. 960 ligt bovendien onder het algemene plafond van 1000, dus deze entry maakt de bewaking hier strenger dan de standaardratchet in plaats van hem op te rekken.

Dat het een pure verhuizing is

De poortwachter heeft de 216 verwijderde regels regel voor regel tegen de 229 toegevoegde gelegd, genormaliseerd op inspringing. Het verschil is exact de kop: commentaarblok, part of, de extension-regel en de sluitende accolade. Geen enkele gewijzigde uitdrukking.

Poorten

make check groen in één keer (9538 tests, dekking 86,9%). Het nieuwe part staat als eigen record in lcov.info (78,8%) — een part met uitvoerbare regels valt dus niet stilletjes buiten de meting — en is in SOURCE_MAP geregistreerd. make check-secrets groen, make sast groen (0 findings).

Sluit #1509. ## Waarom `document_editor_screen.dart` stuitte tijdens #1505 en #1506 twee keer op zijn regelplafond, en beide keren kostte een toevoeging van drie regels eerst een verhuizing. Dat is een teken dat het scherm te veel doet, niet dat het plafond te laag staat. ## Wat De drie weergavestanden — bron naast weergave, het document op pagina's, en het visuele schrijfvlak — plus hun toebehoren (`_wysiwygEditor`, `_editor`, `_preview`) staan nu in `lib/widgets/parts/document_editor_layouts.dart`, als `extension _DocumentEditorLayouts on _DocumentEditorScreenState`. Zelfde library, zelfde staat, zelfde gedrag; het scherm houdt de staat en de bedrading over. Van **1168 naar 953 regels**. De basislijn in `tool/check_conventions.dart` zakt mee van 1237 naar 960 — een plafond dat driehonderd regels boven de werkelijkheid hangt bewaakt niets. 960 ligt bovendien onder het algemene plafond van 1000, dus deze entry maakt de bewaking hier strenger dan de standaardratchet in plaats van hem op te rekken. ## Dat het een pure verhuizing is De poortwachter heeft de 216 verwijderde regels regel voor regel tegen de 229 toegevoegde gelegd, genormaliseerd op inspringing. Het verschil is exact de kop: commentaarblok, `part of`, de `extension`-regel en de sluitende accolade. Geen enkele gewijzigde uitdrukking. ## Poorten `make check` groen in één keer (9538 tests, dekking 86,9%). Het nieuwe part staat als eigen record in `lcov.info` (78,8%) — een part met uitvoerbare regels valt dus niet stilletjes buiten de meting — en is in SOURCE_MAP geregistreerd. `make check-secrets` groen, `make sast` groen (0 findings).

brenno added 1 commit

2026-08-17 10:12:30 +00:00

refactor(documentmodus): de drie weergavestanden in een eigen part

scans / scans (pull_request) Successful in 2m53s

Details

static-gate / static-gate (pull_request) Successful in 6m3s

Details

fc87107793

Het editorscherm stuitte twee keer op zijn regelplafond en elke toevoeging
kostte eerst een verhuizing — het teken dat het scherm te veel deed, niet
dat het plafond te laag stond.

De drie standen (bron naast weergave, het document op pagina's, het
visuele schrijfvlak) en hun toebehoren staan nu in een eigen part, als
extensie op de staat van het scherm. Zelfde library, zelfde gedrag; het
scherm houdt de staat en de bedrading. Van 1168 naar 953 regels.

De basislijn zakt mee naar 960 in plaats van op 1237 te blijven staan: een
plafond dat driehonderd regels boven de werkelijkheid hangt bewaakt niets.

Sluit #1509.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

brenno merged commit 6016ae4a25 into main

2026-08-17 10:18:43 +00:00

brenno referenced this pull request from a commit

2026-08-17 10:18:44 +00:00

refactor(documentmodus): de drie weergavestanden in een eigen part (#1515)

brenno referenced this pull request

2026-08-17 10:46:40 +00:00

document_editor_screen.dart zit krap onder zijn regelplafond #1509