kleidos/test/ui/test_unlock_feedback/ at main · jmrp.io/kleidos

[READ-ONLY] Mirror of https://github.com/jmrplens/kleidos. Kleidos — Hardware BLE password manager for ESP32-S3 (M5StickC Plus2, StickS3, M5Stack Gray, T-Deck, Cardputer) jmrplens.github.io/kleidos/

Fork 0

kleidos / test / ui / test_unlock_feedback /

at main 1 file

José M. Requena Plens Adopt the sheet status-container anatomy for filled status surfaces 5w ago

9bf3bd66

test_unlock_feedback.cpp

Adopt the sheet status-container anatomy for filled status surfaces The unlock result disc was a solid saturated fill. Its foreground had already flip-flopped once (a light glyph measures 1.52:1 on kSuccess and 2.10:1 on kWarning; the interim fix knocked the glyph out in kBg), and an adversarial audit settled the question against the design sheets: they never place any foreground on a saturated status fill. Their anatomy for every status surface is a dark container + a 1px saturated ring + light or saturated ink — so that is now the token vocabulary. New container tokens color.bg.success #0D3B1E / color.bg.warning #1A0E0E (theme::kSuccessFill / kWarningFill), taken verbatim from the sheets. The result disc becomes container fill + saturated ring + kText glyph: the glyph sits at 9.57:1 / 14.29:1 (nominal sRGB), the ring carries the >=3:1 silhouette against kBg including under simulated protanopia (11.06:1 / 6.63:1) — the audit killed a brighter-container proposal precisely because its failure disc fell to 2.65:1 for protanopes. Owner-approved from on-glass Stick captures of both discs. The same audit exposed two full-screen AA failures fixed here: the vault-wipe countdown and the low-battery screen drew kText on saturated kWarning at 2.10:1 — and on e-paper both tones map to INK, so the most consequential copy in the product was literally black-on-black. Both now use the container fill with the title in the status hue. Docs follow: accessibility.md contrast table recomputed from the token hexes (7 of 9 rows were wrong, status.warning was understated 5.8 vs 7.57), the new tokens documented and mirrored (skill + handoff + bundle + web :root), handoff-firmware.md rewritten as a current-state alignment record (it still proposed additions to a file path that no longer exists), and stale src/ui/Theme.h references fixed. Claude-Session: https://claude.ai/code/session_01XBVa8G5jrkprye4gVAkQSU

1 month ago

Configure Feed

Configure Feed