fix(cockpit): tekst van het glas af — uitleesvenster in de flank, label eronder #1929
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!1929
Loading…
Reference in a new issue
No description provided.
Delete branch "feat/cockpit-flank-layout"
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?
Samenvatting
De cockpit-dia zette de waarde-uitlezing als één ongebonden regel "getal+eenheid" ín de wijzerplaat, op de hoogte van de schaalcijfers en de bandeinden. Een eenheid als "% van maximale hartslag" liep dwars door "50", "100" en tot op de bezel; klassiek kapte hem aan de kaartrand af; de HTML-export liet hem het buurinstrument in lopen. Het label zat vast op 18 px, ook op 1920 px breed, en de flanken naast de ronde bezel bleven leeg.
Herontwerp (ontwerp en besluiten:
docs/design/COCKPIT_LAYOUT.md, mock-up op de canvas "Cockpit herontwerp"):lib/services/cockpit_layout.dart, Flutter-vrij) voor painter én HTML-export; elke passingsbeslissing op tekenaantal, niet op gemeten pixels, zodat de export dezelfde regelval kiest. De export tekent daardoor ook dezelfde thermometer en horizon als de app; zijn paneel heeft de verhouding van het app-paneel. De clip-ids in de export dragen een suffix per SVG: twee cockpit-dia's in één document deelden#cockpit-horizon-0en de tweede verloor zijn tekst.Bestandsformaat onaangeroerd; geen nieuwe vertaalstring (de horizonregel "P {pitch} B {bank}" wordt op de dubbele spatie gesplitst).
Tests
test/cockpit_layout_test.dartpint de rekenkern: kolomregel, venster buiten de bezel voor 1–6 meters, de maten uit het ontwerp, en dat de zes meters uit de aanleiding zonder ellipsis en zonder krimp passen; plus de cascade en het kompas met een lange markerregel._valueTexthad geen breedtegrens en geen ellipsis), dus er was niets meetbaars dat "loopt over de schaal" hard kon maken behalve een pixelvergelijking. De goldentype_cockpitis opnieuw uit de renderer gebouwd.cockpit_heading_readouts_test.dart(#1110) gaat op in de nieuwe geometrietest; de exporttests zijn bijgewerkt op de nieuwe tekstknopen en het nieuwe palet.Bewaker
Overgeslagen, expliciet: de wijziging raakt het bestandsformaat, de opslag, afhankelijkheden, uitgaand verkeer noch een publieke belofte. Het
cockpit-blok blijft byte voor byte gelijk.Wat verandert voor bestaande decks
Poort
make checklokaal groen (zie de commit-status hieronder als de statische poort ook groen is).make check-secretsenmake sast: zie de laatste reactie op deze PR.🤖 Generated with Claude Code
Poortresultaten op HEAD van deze tak:
make check: groen (formatting, analyse, conventies, privacyprojectie, methodelengte, dode code, hardgecodeerde tekst, commentaartaal, volledige Flutter-suite, dekkingsvloeren, golden-suite op macOS).make check-secrets: gitleaks en trufflehog, 0 geheimen (werkboom en historie).make sast: semgrep met de lokale regels, 4 regels over 1197 bestanden, 0 bevindingen.Stand na de beeldkeuring in de echte renderer (twee rondes, headless harnas met EB Garamond op 2×, plus Chrome voor de export), kop
880f10045:Eerste ronde vond acht punten, alle verholpen en in de tweede ronde met schermafdruk bevestigd: export-animatie die de cellen op elkaar zette (
transform:scale()in de CSS-keyframes overschreef hettransform-attribuut), rollend getal met decimalen buiten het venster, thermometerbuis door de lampjes, naald door de "0" van klim/daling, horizonregels buiten de plaat, onzichtbaar vliegtuigsymbool (2,5:1), schaalcijfers onder AA, vlekjes in de miniatuur. Tweede ronde: vier restpunten (getalmaat per cel in gestapelde modus, export-schaalcijfers op 85 % dekking, inline-eenheid onder de tekendrempel, koppelteken als minteken), verholpen en met tests vastgelegd.make checkop880f10045: groen (volledige suite, dekkingsvloeren, golden-suite).make check-secretsenmake sast: 0 bevindingen.Buiten de scope van deze PR, gezien in de keuring: het LibreKAT-logo (donkere inkt) is op een donkere dia-achtergrond vrijwel onzichtbaar, en de logo-vrije zone laat onder het paneel ±190 px leeg over de volle breedte.