docs(design): het callout-formaat na de tweede review (#1801) #1806
No reviewers
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!1806
Loading…
Reference in a new issue
No description provided.
Delete branch "docs/1801-callout-ontwerp-v4"
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?
Verwerkt de tweede reviewronde op
docs/design/IMAGE_CALLOUTS.md(issue #1801). Alle zeven punten zijn beantwoord; twee ervan veranderden een besluit in plaats van een formulering.Twee besluiten die omgingen
marp --theme-setde markeringen tóch tekende. Dat is een tweede kopie van een feit dat de front matter al vastlegt, en een handmatige bewerking van het.mdkan die kopie stil laten liegen.v0.4.10is op 25 augustus uitgebracht (niet-concept, 12 assets) env0.4.9kreeg nooit een release. De bewaarproef noemt daarom geen versie meer bij naam maar lost 'nieuwste uitgebrachte tag' op bij de poort, en is een uitvoerbare kruisversietoets in plaats van een weggegooide probe. De conclusie zelf houdt stand: de diff over de vier dragerpaden tussenv0.4.10enmainis leeg.De waardenbotsing, opgeschreven (verplicht want dit raakt het bestandsformaat; bewaker gedraaid)
Uitwisselbaarheid tegen een rijke functie, en uitwisselbaarheid wint — dezelfde uitkomst als in §11, nu ook voor de opslagvorm. Een vreemd gereedschap dat niets tekent is eerlijk; een dat het verkeerde tekent niet, want de auteur kan het verschil niet zien. De prijs is benoemd:
marp --theme-settoont voortaan de indeling, de tekst en(A), en geen markeringen. Wat dit besluit zou terugdraaien: een drager waarvan de afgeleide kopie niet kan verouderen, of waarvan het verouderd-zijn door het lezende gereedschap te zien is. CSS kan geen van beide.Gemeten in plaats van beloofd
De geometrie was in revisie 3 alleen voor het gecentreerde cover-geval bewezen. Nu volledig (§4.1) en gemeten in headless Chrome (§4.2): 486 gevallen — 3 beeldverhoudingen × 3 slots × zoominvoer 0/100/140/400/−50/5000 × focaal 0/0,5/1 op beide assen × 8 doelen op elke hoek en randmidden plus een regio. Grootste afwijking 0,03 px.
Twee formuleringen die logisch klopten bleken fout, en staan nu als contract in het document:
max-width/max-height+aspect-ratio— tot 2144 px mis. De richting wordt bij generatie gekozen.position: relativeop de beelddoos vallen de markeringspercentages terug op de zoomdoos — 1184 px mis, terwijl de afbeelding zelf goed staat.Bijvangst: een bestaand gebrek (§4.3)
Bij het meten van de zoomtak bleek dat elke zoom boven 100% hetzelfde beeld oplevert als 100%:
Alignversoepelt de grenzen maar houdt het slot als maximum, enSizedBoxdwingt zijn maat daarbinnen af, dus de vergrote doos wordt teruggeknepen. Gemeten metflutter testop de exacte widgetketen, en het proefbeeld in de bijsnijddialoog knijpt op dezelfde manier — vandaar dat editor en dia het netjes met elkáár eens zijn. Dat krijgt een eigen issue en staat als voorwaarde vooraf naast #1803 in de werkvolgorde; zonder die reparatie zou de eerste surface die §4.1 getrouw uitvoert de markeringen ergens anders neerzetten dan Flutter.De overige punten
ocideck_calloutsbezit: de bescherming die de oude lezer het blok laat behouden bestaat alleen omdat hij de sleutel niet bezit.(A)een verwijzing is en wanneer gewoon proza, inclusief de geplakte dubbele bullet.ZZterwijl het plafond 26 was; nu één letter.Poorten
make check-static(exit 0),make check-secrets(gitleaks + trufflehog, werkboom én historie),make sast(semgrep, 0 bevindingen),check-translated-mermaid,translate-docs-check,check-comment-language,check-audience-boundary— alle groen. De wijziging raakt geen Dart, dus de testsuite en de dekkingsvloer zijn niet aan de orde;docs/design/**is bewust ongebundeld en heeft geen registratie nodig.