fix(iconen): Linux en Windows droegen nog het logo van vóór de rebrand #765
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!765
Loading…
Reference in a new issue
No description provided.
Delete branch "fix/platform-iconen-huisstijl"
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?
Linux en Windows droegen nog het logo van vóór de rebrand.
Wat er mis was
Toen het merk in juni van de fotokat naar de lijntekening ging (
da7b7c9e→dc62e03f→c6ccd42c), zijn alleen de macOS-set en de webiconen opnieuw gegenereerd. Deze twee bleven staan op wat20906ddber in juni had neergezet:linux/runner/resources/app_icon.png— ingebouwd viaCMakeLists.txt:100enmy_application.cc:28windows/runner/resources/app_icon.ico— viaRunner.rc:55Allebei ondersteunde bouwdoelen. Wie
make build-linuxofmake build-windowsdraaide, kreeg een app met de oude De Winter-huisstijl in de taakbalk.De behandeling
Dezelfde die de macOS-set kreeg in
c6ccd42c: bijsnijden tot de tekening, schalen tot 87,7% van de canvashoogte, centreren op wit. Ik heb dat recept eerst tegen het bestaande macOS-icoon gehouden voor ik het toepaste — het reproduceertapp_icon_512.pngop een handvol antialiasingpixels na (RMSE 4,3%), dus het is aantoonbaar dezelfde behandeling en niet een nieuwe. De.icohoudt al zijn zeven maten (16/24/32/48/64/128/256). Naast elkaar gelegd zijn de nieuwe 16 en 32 niet te onderscheiden van wat macOS al toont.Ondoorzichtig wit, geen transparantie. Het huidige Linux-icoon hád transparantie, maar dat werkte alleen omdat de foto zijn eigen kleur meebracht. Het merk is donkere inkt: op een donkere taakbalk verdwijnt die zonder witte plaat — de spiegelklacht van #735.
Waarom geen enkele poort dit zag
Dat is het eigenlijke gebrek, en daar zit de test op. Iconen zijn geen Dart, staan niet in
pubspec.yamlen worden door de runner via een pad opgepikt; er was niets dat rood werd als er een achterbleef.test/platform_icon_branding_test.dartvergelijkt daarom de tekening: bijsnijden tot de inhoud, platslaan tot een grijswaarde-vingerafdruk van 32×32, en die van elk bureaubladdoel naast het merk leggen. Plus een kleurtoets over élk formaat — de fotokat is voor de helft verzadigde kleur, de lijntekening is neutrale inkt.Eerst rood gedraaid tegen de ónherstelde iconen, daarna pas groen. De toets loopt bewust over álle vier de doelen, niet alleen de twee die ik repareerde: de fout was "één platform vergeten", en een test die alleen deze twee kent had de volgende omissie weer gemist.
De kleine maten (16–64) doen niet mee aan de tekeningtoets — daar waaieren de dunne lijnen zo ver uit dat de vingerafdruk vanzelf richting 0,19 loopt, óók bij een volstrekt correct icoon. Dat is verschaling, geen huisstijl; die maten worden op kleur getoetst.
Documentatie
Er is geen generatorscript; de sets worden met de hand met ImageMagick uitgeschreven en gecommit. Dat stond nergens, en precies daarom kon de vorige rebrand er twee vergeten.
docs/BUILD.mdheeft nu een sectie App icons met de twee commando's, de inkaderingsregel en de reden voor ondoorzichtig wit.Buiten scope, met opzet
ios/enandroid/zijn geen ondersteunde bouwdoelen — geen Makefile-doel, README noemt ze niet,docs/BUILD.mdevenmin. Hun icoonsets dragen nog wél het oude logo. Dat staat nu als kanttekening in BUILD.md; wat ermee gebeurt is een aparte keuze.Poorten
make check— exit 0, alle poorten groen. Opnieuw gedraaid ná de rebase op204cfcc7.make check-secrets— gitleaks en trufflehog schoon, werkboom én historie.make sast— semgrep 0 bevindingen.make sbom: geen enkele afhankelijkheid gewijzigd.De bewakerstap heb ik overgeslagen: dit raakt geen bestandsformaat, opslag, afhankelijkheid, uitgaand verkeer of publieke belofte.
🤖 Generated with Claude Code