enhancement(quality): hoe dia's er ná auto-reparatie uitzien — vervolgtitels, dode split-knop, dichtheidsklif #1164
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#1164
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?
Voortgekomen uit de beeldkeuring bij #1159 (de font-fix zelf is los opgelost en gemerged). Een herbeoordeling van hoe dia's er ná 'Fix alle problemen' / splitsen uitzien, met eigen ogen over strook, presentatiemodus en PDF-export. Deze punten staan los van de font-fix en zijn deels al bestaand gedrag.
Bevindingen (naar impact)
Vervolgpagina's herhalen exact dezelfde titel, zonder markering. Na een split draagt elke vervolgpagina identiek dezelfde H1 als de eerste — geen '(vervolg)', '(2/3)' o.i.d. Bij een titel met een aantal erin ('Aantal-overloop (achttien korte bullets)') staat dat verkeerde aantal dan 3× op pagina's van 6 bullets. Zichtbaar in slidestrook, presentatie én export. Bestaand gedrag (geldt ook voor handmatig splitsen), maar het oogt onverzorgd. Klein codewerk, breed oppervlak (preview/presenter/publiek/export) — vraagt visuele keuring op alle vier + goldens. Productkeuze: wat toont een vervolgpagina als titel?
'Fix alle problemen' is een stille no-op op woordigheid-alleen dia's. Een dia die leesbaar rendert (> 70%) maar te veel woorden per bullet heeft (bulletWordCountCritical / bulletAverageLengthHigh) wordt door geen enkele automatische fix aangeraakt: te weinig bullets voor de aantal-split, geen dichtheidsmelding voor de nieuwe gate, en woord-count-meldingen worden niet auto-opgelost. De gebruiker ziet een rode badge en 'Fix alle problemen' doet er niets mee. Overweeg deze gevallen duidelijker naar 'Uitleg naar notities' te leiden (splitsen geneest woordigheid niet).
Restwaarschuwingen na een split + dode knop. Gesplitste pagina's van lange bullets dragen nog woord-count-waarschuwingen die verder splitsen niet oplost (raakt de kMinPageBullets-vloer). Toch verschijnt er een 'Splits slide'-knop die niets meer uitricht. Bied daar 'Uitleg naar notities'/herschrijven aan i.p.v. een actie die niets doet.
De 70%-klif is onzichtbaar en spreekt de woord-melding tegen. Of een lange-bullet-dia splitst hangt op precies 0,70 fit-scale. Twee even-vol ogende dia's (72% vs 64%) krijgen tegengestelde behandeling. Overweeg een zachtere band of de drempel gelijk te trekken met de woord-count-melding.
[Apart] Ontbrekend thema-logo rendert als 'Bestand niet gevonden'-placeholder linksonder op elke dia (editor, presentatie, export). Buiten deze scope; eigen ticket waard — het degradeert elke geëxporteerde pagina. (Mogelijk configuratie/omgeving; eerst verifiëren.)
Advies
Punt 1 is de meest zichtbare verbetering aan 'hoe dia's er na reparatie uitzien' en de sterkste kandidaat om als eerste op te pakken, maar het raakt de rendering op alle oppervlakken en is deels een productkeuze — daarom hier vastgelegd i.p.v. in de release-cyclus meegesleept. Punt 5 verdient bij bevestiging een eigen issue.
Opgepakt. Ik verifieer eerst punt 5 (logo: bug of omgeving) en de haalbaarheid van punt 1 (vervolgtitels), en pak dan de concreet-veilige verbeteringen aan. Punt 1 vraagt mogelijk een productkeuze over de titelvorm.
Punt 4 (de 70%-klif) laat ik bewust liggen in deze ronde: de drempel verschuiven of verzachten is een risicovolle, deels-productmatige wijziging die de split-beslissing over het hele product raakt, en de winst is discutabel — twee even-vol ogende dia's krijgen nu tegengestelde behandeling, maar een zachte band introduceert zijn eigen willekeur. Beter apart afwegen dan meesleuren. Punt 5 (logo) is afgesplitst als eigen issue (waarschijnlijk omgeving, met een defensieve exportverbetering als kandidaat).
Opgelost op main via PR #1176 (merge
6e3f55b5). Aangepakt:Niet in deze ronde:
Cosmetisch restpunt (niet-blokkerend, uit de keuring): bij een titel over meerdere regels drijft de teller naar de rechterrand.
Opgevolgd: het cosmetische restpunt (teller zweefde naar de rechterrand bij een titel over meerdere regels) is opgelost in PR #1179 (merge
710340e5). De teller loopt nu als inline trailing-span in de titelparagraaf mee en plakt met een vaste-spatie aan het laatste woord — visueel bevestigd over strook, presentatie, volledige-deck-overzicht en PDF, bij lange én korte titels.