fix(privacy): dia-thumbnail-badge noemt de wérkelijke privacy-stand (#1112) #1115
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!1115
Loading…
Reference in a new issue
No description provided.
Delete branch "fix/privacy-badge-disposition-1112"
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?
Waarom
De privacy-badge op de dia-thumbnails toonde bij aanwijzen altijd "Persoonsgegevens geaccepteerd" — ook op een dia die op weglaten (redact) of accepteren + waarschuwen (shield) stond. De grijze badge dekt drie dispositie-standen af (
isResolved= "nietwarn"), maar noemde er maar één. "Geaccepteerd" op een weggelaten dia is onwaar: daar wordt niets geaccepteerd, het wordt juist weggehaald. Dat verklaart de melding "soms is het geaccepteerd en soms niet" (#1112).Wat
De effectieve dispositie gaat nu mee naar de badge-tekst, die per stand het juiste woord kiest — in de geest van het instellingenpaneel (
privacyDispositionShortLabel): geaccepteerd en weggelaten zijn er letterlijk uit, gemarkeerd voor de ontvanger is een preciezere parafrase van het paneelwoord gewaarschuwd.acceptshieldredactwarn)Zowel de tooltip als het schermlezer-label lezen dezelfde functie, dus ook de voorgelezen tekst klopt nu. De badge-kleur verandert niet (grijs blijft "afgehandeld"); alleen de tekst noemt wélke van de drie.
Alleen
lib/widgets/slides/slide_thumbnail.dart; de effectieve dispositie wordt inbuildgelezen en doorgegeven,_badgeToneskrijgt de resolved-vlag als parameter.Test
Regressietest per dispositie in
test/slide_badge_gestures_test.dart(groep "de badge noemt de wérkelijke privacy-stand"), inclusief de kern van de bug: eenredact-dia toont "weggelaten", níét "geaccepteerd". De bug was bovendien empirisch vastgelegd in de bestaande test "dubbelklik laat redactie met rust", die op main "Persoonsgegevens geaccepteerd" verwachtte op een weggelaten dia — die verwachting is nu "weggelaten".l10n
Twee nieuwe bronstrings ("Persoonsgegevens weggelaten", "Persoonsgegevens gemarkeerd voor de ontvanger") in alle 31 niet-Nederlandse overlays via
make add-l10n.Documentatie
CHANGELOG (Fixed) en
docs/USER_GUIDE.md: de grijze privacy-badge betekent "afgehandeld" (geaccepteerd, gemarkeerd voor de ontvanger, of weggelaten), niet enkel "geaccepteerd", en de badge noemt nu welke van de drie.Poorten
make checkgroen (statische poorten + volledige testsuite 8064 tests + dekking 87,6% + per-bestand-vloer).make check-secrets(gitleaks + trufflehog): 0 geheimen.make sast(semgrep, lokale regels): 0 bevindingen.Bewaker
Interface- en documentatietekst over hoe persoonsgegevens behandeld worden = een privacy-belofte in de UI, dus de bewaker heeft meegelezen. Oordeel: akkoord — de bewoording is eerlijk in alle drie de standen en verwijdert een valse geruststelling (shield/redact zeiden ten onrechte "geaccepteerd"). Eén bevinding verwerkt: de claim "in de woorden van het instellingenpaneel" is verzacht naar "in de geest van", want voor shield gebruikt het paneel "gewaarschuwd" en is de badge-term een parafrase.
Buiten de rol van de bewaker, genoteerd voor de privacyexpert (los van deze fix, niet erdoor geïntroduceerd): of een louter-beeld-bevinding met stand
redact/shieldook echt door de beeld-privacypijplijn wordt nageleefd — de badge spiegelt de dispositie, net als voorheen.Closes #1112
4ec54a31702950eb9b38