document_editor_screen.dart zit krap onder zijn regelplafond #1509

New issue

Closed

opened 2026-08-17 09:40:42 +00:00 by brenno · 2 comments

brenno commented

2026-08-17 09:40:42 +00:00

Owner

Na PR #1506 is lib/widgets/document_editor_screen.dart 1168 regels met een plafond van 1237. Tijdens die tak stond hij twee keer exact óp of één regel boven het plafond, en is er twee keer code uitgetild om eronder te blijven (de Overzicht-rail, de paginamaat-indicator, en de bronbewerkingen naar een eigen part).

Dat is een teken dat het scherm te veel doet, niet dat het plafond te laag staat. Het bevat nu: drie weergavestanden, de werkbalk-bedrading, opslaan/exporteren, stijlbeheer, invoegen van blokken, de grafiek- en tabelbewerking, en de synchronisatie tussen bron en visuele cursor.

Voorstel: de standen (_visualLayout, _sourceLayout, _pagesLayout) en hun toebehoren naar eigen bestanden, zodat het scherm alleen nog de staat en de bedrading houdt. Daarna de basislijn verlagen naar de nieuwe werkelijkheid.

Geen haast, wel voordat iemand er de volgende functie in bouwt: nu kost elke toevoeging eerst een verhuizing.

Na PR #1506 is `lib/widgets/document_editor_screen.dart` 1168 regels met een plafond van 1237. Tijdens die tak stond hij twee keer exact óp of één regel boven het plafond, en is er twee keer code uitgetild om eronder te blijven (de Overzicht-rail, de paginamaat-indicator, en de bronbewerkingen naar een eigen part). Dat is een teken dat het scherm te veel doet, niet dat het plafond te laag staat. Het bevat nu: drie weergavestanden, de werkbalk-bedrading, opslaan/exporteren, stijlbeheer, invoegen van blokken, de grafiek- en tabelbewerking, en de synchronisatie tussen bron en visuele cursor. **Voorstel:** de standen (`_visualLayout`, `_sourceLayout`, `_pagesLayout`) en hun toebehoren naar eigen bestanden, zodat het scherm alleen nog de staat en de bedrading houdt. Daarna de basislijn verlagen naar de nieuwe werkelijkheid. Geen haast, wel voordat iemand er de volgende functie in bouwt: nu kost elke toevoeging eerst een verhuizing.

brenno added the

enhancement

in-progress

labels

2026-08-17 09:40:42 +00:00

brenno commented

2026-08-17 09:42:01 +00:00

Author

Owner

Opgepakt op tak refactor/documentscherm-standen.

Opgepakt op tak `refactor/documentscherm-standen`.

brenno referenced this issue from a commit