Kwaliteitspaneel: aanbevolen 'Splits slide' laat het aantal waarschuwingen exploderen #1289
Labels
No labels
accepted
bug
declined
docs
duplicate
enhancement
good first issue
in-progress
needs-info
privacy
security
triage
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
LibreKAT/Ocideck#1289
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Waarneming
Gevonden tijdens de beeldkeuring van #1279. Op een overvolle bullets+afbeelding-slide (Proefpresentatie.md slide 5, ~59 volzin-bullets) beveelt het kwaliteitspaneel 'Splits slide' aan. Voer je die aanbevolen actie uit, dan gaat het deck van 7 fouten / 7 waarschuwingen / 2 tips naar 5 / 42 / 7: elke deel-slide krijgt 2-3 tekstdichtheid-waarschuwingen ('verkleind tot X%', 'veel woorden (127-142)', 'lange bullets (26-51 woorden per bullet)').
Waarom dit wringt
De gebruiker volgt de aanbeveling en wordt beloond met zes keer zoveel meldingen. De meldingen zijn op zich niet onjuist — de bullets zijn lange volzinnen — maar 'Splits slide' lost de dichtheid-per-aantal op, niet de lengte-per-bullet. Het voelt als straf op gehoorzaamheid.
Mogelijke richtingen (te wegen, niet besloten)
Raakt geen bestandsformaat of publieke belofte; puur kwaliteitspaneel-UX. Verwant aan #1280 (dat de 'Fix alle problemen'-knop eerlijk maakte).
Opgepakt. Tak: fix/1289-quality-split-warnings (worktree ocideck-1289, op verse origin/main).
Richting: ik werk direction 1 uit — na 'Splits slide' de lengte-gedreven dichtheidsmeldingen (veel woorden, lange bullets, meerzins, verkleind) die over de pagina's van één split-run heen herhalen samenvatten tot één melding per soort per reeks, in plaats van per pagina. Splitsen lost het aantal-per-dia op; deze meldingen komen uit de lengte van de bullets en vermenigvuldigen anders over de deelpagina's. De telling is letterlijk het aantal meldingobjecten, dus dit gebeurt op de weergavelaag (deck_quality_provider), niet in de analyzer die de fix-motor voedt — die blijft de grondwaarheid zien. De collapse pakt alleen echte split-runs (continuesSplit-keten), nooit los geschreven bullet-dia's, en nooit fouten (critical blijft per pagina zichtbaar).
Verwachte reikwijdte: lib/state/deck_quality_provider.dart, een kleine nieuwe pure helper onder lib/services/slide_quality/, lib/widgets/panels/slide_quality_panel.dart (locatielabel 'Slides X–Y'), l10n voor 1–2 nieuwe strings, tests + CHANGELOG.
Gemerged in main via #1293 — commit
c0955dc5(merge5bd44569).Richting 1 uitgewerkt: de lengte-gedreven dichtheidsmeldingen (veel woorden, gemiddeld lange bullets, meerzins, diepe nesting, 'verkleind tot X%') die over twee of meer pagina's van dezelfde split-run heen herhalen, worden op de weergavelaag samengevat tot één melding per soort per reeks, met de tekst 'Geldt voor {n} slides van deze gesplitste reeks'. De ruwe analyse (deckQualityRawProvider), de fix-motor ('Los automatisch op wat kan') en de per-dia-badges op de slidestrook blijven elke pagina apart zien; fouten (critical) en de aantal-gedreven melding 'te veel bullets' blijven per pagina staan, want die lost verder splitsen wél op.
Nieuwe pure functie lib/services/slide_quality/split_run_density_summary.dart met regressietests (pure eenheidstests + echte-analyse-reproductie + provider-test dat ruw explodeert terwijl de weergave samenvat). 31 vertalingen toegevoegd. make check groen op de gerebasede staat (8654 tests).
Niet meegenomen: een run-brede fix-actie die de hele reeks in één klik aanpakt (bv. 'uitleg naar notities' over alle pagina's). De bestaande per-pagina-acties blijven op de samengevatte melding beschikbaar; een reeks-brede variant kan later als losse verbetering.