LibreKAT/Ocideck

Watch

Fork 1

Paginaweergave telt eindnoten niet mee in de gemeten documenthoogte #1653

New issue

Closed

opened 2026-08-21 12:00:26 +00:00 by brenno · 1 comment

brenno commented

2026-08-21 12:00:26 +00:00

Owner

Probleem

Bij footnotePlacement=document voegt DocumentMarkdownView de eindnotenlijst buiten de blockWrapper toe. PagedDocumentView meet alleen blockTexts en krijgt daardoor geen hoogte voor de eindnoten. Lange eindnoten kunnen voorbij de laatste berekende pagina vallen en worden afgeknipt of onbereikbaar.

Reproductie

Kies voetnoten achterin het document.
Voeg genoeg/lange voetnoten toe om een extra pagina te vullen.
Open de paginaweergave.

Verwacht

De eindnoten tellen als gewone documentblokken mee en veroorzaken zo nodig extra pagina's.

Technische aanwijzing

lib/widgets/reader/paged_document_view.dart:_measure gebruikt blockCount = _blockTexts.length en footnotesAtEnd=true, maar alleen widgets via blockWrapper leveren gemeten hoogtes.

Gevonden bij audit van commit d439638c6b.

## Probleem Bij footnotePlacement=document voegt DocumentMarkdownView de eindnotenlijst buiten de blockWrapper toe. PagedDocumentView meet alleen blockTexts en krijgt daardoor geen hoogte voor de eindnoten. Lange eindnoten kunnen voorbij de laatste berekende pagina vallen en worden afgeknipt of onbereikbaar. ## Reproductie 1. Kies voetnoten achterin het document. 2. Voeg genoeg/lange voetnoten toe om een extra pagina te vullen. 3. Open de paginaweergave. ## Verwacht De eindnoten tellen als gewone documentblokken mee en veroorzaken zo nodig extra pagina's. ## Technische aanwijzing lib/widgets/reader/paged_document_view.dart:_measure gebruikt blockCount = _blockTexts.length en footnotesAtEnd=true, maar alleen widgets via blockWrapper leveren gemeten hoogtes. Gevonden bij audit van commit d439638c6bd1b519d68d87680cb022b7d5eddc85.

brenno added the

bug

triage

labels

2026-08-21 12:00:26 +00:00

brenno added

accepted

and removed

triage

labels

2026-08-21 22:52:13 +00:00

brenno commented

2026-08-21 22:52:13 +00:00

Author

Owner

Triage: accepted

Bevestigd tegen main (e93ef205c). lib/widgets/reader/document_markdown_view.dart hangt _footnoteList(...) buiten _wrapped(...) in de kolom. blockWrapper ziet die lijst dus nooit, en PagedDocumentView._measure telt alleen _blockTexts.length plus (bij noten-op-de-pagina) de losse noten. Bij footnotePlacement=document is _notesOnPage onwaar, gaat footnotesAtEnd: true mee, en heeft de eindnotenlijst geen enkele gemeten hoogte. De paginaverdeling weet niet dat die tekst er is.

Oplossingsrichting

De eindnotenlijst een eigen blokindex geven en door blockWrapper sturen: _wrapped(blocks.length, _footnoteList(...)). Dan meet de paginaweergave hem als elk ander blok en volgt de paginering vanzelf. blockCount in _measure/_onMeasured wordt blocks.length + 1 wanneer footnotesAtEnd én er noten zijn.

Alternatief — de lijst apart meten in de paginaweergave — is slechter: dan staat dezelfde opmaak op twee plekken en lopen scherm en druk opnieuw uit elkaar.

Nuance die in het werk hoort: één blok voor álle eindnoten betekent dat de lijst alleen op blokniveau over vellen breekt. documentPageOffsets verdeelt een te hoog blok al over meerdere vensters (lib/services/document_pagination.dart, de tak height + reservedFor(index) > pageHeight), dus de noten blijven leesbaar. Per noot een blok is netter en is een grotere ingreep; niet nu.

Regressietest (verplicht)

test/paged_document_view_test.dart: footnotePlacement=document met genoeg noten voor een extra vel → het aantal vellen groeit mee en de laatste noot valt binnen het laatste vel.

Kosten

Twee bestanden (document_markdown_view.dart, paged_document_view.dart), plus test. Controleer of DocumentMarkdownView.blockTexts mee moet groeien — de zoek-/ankerindexen hangen aan dezelfde nummering.

Prioriteit

Middelhoog: inhoud verdwijnt zonder melding uit beeld.

## Triage: accepted **Bevestigd tegen `main` (e93ef205c).** `lib/widgets/reader/document_markdown_view.dart` hangt `_footnoteList(...)` buiten `_wrapped(...)` in de kolom. `blockWrapper` ziet die lijst dus nooit, en `PagedDocumentView._measure` telt alleen `_blockTexts.length` plus (bij noten-op-de-pagina) de losse noten. Bij `footnotePlacement=document` is `_notesOnPage` onwaar, gaat `footnotesAtEnd: true` mee, en heeft de eindnotenlijst geen enkele gemeten hoogte. De paginaverdeling weet niet dat die tekst er is. ## Oplossingsrichting De eindnotenlijst een eigen blokindex geven en door `blockWrapper` sturen: `_wrapped(blocks.length, _footnoteList(...))`. Dan meet de paginaweergave hem als elk ander blok en volgt de paginering vanzelf. `blockCount` in `_measure`/`_onMeasured` wordt `blocks.length + 1` wanneer `footnotesAtEnd` én er noten zijn. Alternatief — de lijst apart meten in de paginaweergave — is slechter: dan staat dezelfde opmaak op twee plekken en lopen scherm en druk opnieuw uit elkaar. Nuance die in het werk hoort: één blok voor álle eindnoten betekent dat de lijst alleen op blokniveau over vellen breekt. `documentPageOffsets` verdeelt een te hoog blok al over meerdere vensters (`lib/services/document_pagination.dart`, de tak `height + reservedFor(index) > pageHeight`), dus de noten blijven leesbaar. Per noot een blok is netter en is een grotere ingreep; niet nu. ## Regressietest (verplicht) `test/paged_document_view_test.dart`: `footnotePlacement=document` met genoeg noten voor een extra vel → het aantal vellen groeit mee en de laatste noot valt binnen het laatste vel. ## Kosten Twee bestanden (`document_markdown_view.dart`, `paged_document_view.dart`), plus test. Controleer of `DocumentMarkdownView.blockTexts` mee moet groeien — de zoek-/ankerindexen hangen aan dezelfde nummering. ## Prioriteit Middelhoog: inhoud verdwijnt zonder melding uit beeld.

brenno added the

in-progress

label

2026-08-21 23:36:52 +00:00

brenno referenced this issue from a commit

2026-08-22 00:18:57 +00:00

fix: tweede batch accepted issues (#1699, #1683, #1673, #1652, #1653, #1655, #1654)

brenno referenced this issue

2026-08-22 00:19:15 +00:00

fix: alle accepted issues in één batch (17 issues) #1706

brenno referenced this issue from a commit

2026-08-22 00:31:53 +00:00