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

jmrp.io / kleidos

0

[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/

0

kleidos / test / ui /

at main 35 folders

José M. Requena Plens Show battery as a fixed-width gauge in the header 4d ago

test_aa_known_bg

Show battery as a fixed-width gauge in the header The status cluster drew the battery as "NN%" text; a full "100%" is four characters and starved the title on the 135 px Stick header, forcing an early ellipsis. Replace it with a fixed-width gauge glyph (outline + terminal + proportional fill, tinted green while charging and red at <=15%). The level reads at a glance from the fill and the title gets the reclaimed width. Removes the now-unused percentage-formatting helper. Claude-Session: https://claude.ai/code/session_01SV58JXhfxhhc9DC6vdjBo4

4 days ago

Animate the PIN dial at ~30 fps (partial + incremental redraw) The dial repainted the whole frame every tick — fillScreen + re-render of the static brand/ref-ring/markers/dots/footer + a full ~22 ms blit — so it ran at ~15 fps and the fill stepped in coarse ~36 deg jumps. Within a digit only the wedge advances; the hero numeral, active marker and ref-ring numeral change only on a digit boundary. - Split drawStickArc into the (static-per-digit) ref-ring and the dial, and add stickDialBox() so a partial repaint clears only the dial's box — the dirty region collapses there and the blit no longer covers the whole panel. - New AA primitive fillAnnulusSectorAA(from,to) (+ host-tested sectorCoverage255) and drawPinDialSlice: a same-digit tick paints ONLY the newly revealed [lastSweepPercent_, percent] arc slice over the existing track (1 deg overlap so the AA seam merges), so the per-tick cost drops from ~67 ms to a few ms. - handleSelecting dispatches: digit boundary / lock -> full drawStickFrame; same-digit advance -> incremental slice (partial-box redraw as fallback). Measured on a Stick S3: a 3 s hold renders ~91 frames (~30 fps, was ~15) with a smooth gradient fill; the static dial is byte-identical. Native +5 sector tests. (Establishes the dirty-only render pattern — repaint only what changes — to roll out across the rest of the animated UI next.)

3 weeks ago

test_action_bar_component

feat(ui): add primitive-drawn icon library + swap affordance stand-ins

1 month ago

fix(char-wheel): stop the ring spinning backward across a lap wrap After the glyph ring passed a full lap (Z -> A), the next single-step tap animated the whole ring backward through the box ("all the letters flash by") instead of stepping one glyph forward. The running ring float grows freely (it is rendered modulo the ring length), so after a wrap its value is e.g. 26 (a 26-glyph ring) while the settled integer index has wrapped to 0. Tween::retarget starts from the tween's CURRENT value, so retargeting to wrappedIndex+1 (= 1) animated 26 -> 1 — a backward sweep of the whole ring. Fix: when re-seeding a fresh slide (the previous one settled), re-base the tween onto the wrapped integer index with Tween::snap before retargeting, so the step is always +1 forward. The snap is to the same on-screen glyph (value mod ringLen), so it is invisible; a continuous hold keeps the tween active and never hits this path. Add a Tween regression test for the re-based forward step.

3 weeks ago

test_brand_mark_component

Migrate brand mark + PIN footer to components, drop legacy widgets Promote the Kleidos placeholder key mark to a public, host-testable ui::drawBrandKeyMark component and route its remaining consumers (pin_entry drawStickLockup, brand_renderer fallback, onboarding welcome fallback) through it instead of the legacy kleidos::ui::brand::drawMark widget. Convert the PIN-entry footer (drawStickFrame / drawInstructions) to ui::drawActionBar in the same file edit. Delete the legacy src/ui/widgets/action_bar.{h,cpp} and brand_mark.{h,cpp}; all consumers now use the stateless components. Extend the brand_mark component test with key-mark ring/cap, mono no-fill, and tiny-radius clamp cases.

3 weeks ago

test_brand_registry

Group brand decode params into structs and modernize casts (S107/S6004/S7035) Reduce decodeEntryToRgb565 and its color/mono helpers below the 7-parameter limit by grouping the atlas source, tint options, scratch buffer, and pixel output into small structs; update the renderer callers and native tests. Fold brand_registry root buffer into an if init-statement and cast the welcome-mark family with std::to_underlying. Claude-Session: https://claude.ai/code/session_01Y7Cy1HetRp6TZAUAeekN8X

2 weeks ago

test_button_component

feat(ui): add Phase 3 catalog components (MenuList/Dialog/Toast/Progress/Button/TextField/TabBar) Add seven stateless reference components to the in-house UI library, each props-in / draw-out over the ui::Canvas seam with a Mono1/e-paper branch and host RecordingCanvas tests across the four canonical geometries: - MenuList / ListRow — scrollable list (absorbs OnDeviceMenu): selection, scroll window, toggle/value/chevron trailers, scroll rail. - Dialog — centered modal/confirm card (title + wrapped body + 1-2 actions). - Toast / Banner — transient overlay + status strip (absorbs popup_system raster half + status_notifier). - ProgressBar / ProgressArc — linear bar + integer-math circular fill arc. - Button — pressable button with visual states (normal/focused/pressed/ disabled/danger). - TextField — single-line text/password field with caret + tail overflow. - TabBar — horizontal tab strip with a selected tab. Gap A: interactive components expose a visual state/selected/focused prop and render each state now; the focus model that drives selection lands later. Gap B: overflow/scroll/long-text handled by bounds math (visible-window + fit-to-rect) since Canvas::clip is advisory — no HW clip added. Additive only: new component sources + suites, plus platformio.ini build/test registrations and a docs/groups.dox group extension.

1 month ago

test_char_grid_component

Rework 2-button text editors to the approved mockups Trace the owner mockups for the CharWheel carousel and CharGrid grid: - Selection box is the top overlay (fill -> border -> glyph), so it never loses its right/bottom border; rounded corners at min(w,h)/5; a translucent navy fill via an RGB565 channel lerp; equal-weight cyan border. - Drop the "= current value" suggestion stop entirely; rings and grids are pure glyphs. - New two-region button model: body A=move/A-hold=fast/B=append/B-hold=switch to the chip selector; selector A=next chip/B=activate (group->body, DEL, OK=commit) /B-hold=back to body. Save is OK-chip only. - Visible slide animation between letters on color panels. Editor unit suites updated for the new ring length and removed suggestion stop.

3 weeks ago

test_char_wheel_component

fix(char-wheel): make the glyph-ring slide actually animate on device The carousel "lens" slide rendered as a near-instant jump on hardware: the char-wheel screen repaints ~14 fps (a full-body region blit is ~72 ms/frame), so the 150 ms ring tween only produced ~2 frames. The renderer was correct (a frozen offset shows the full lens) but the live drive never got enough frames. Fix, two parts: - Dirty-only slide path: while the ring is sliding the chips and label do not change, so repaint and blit ONLY the wheel band (new charWheelBandRect / drawCharWheelBand). That roughly halves the per-frame cost. The per-tap advance no longer forces a full-body repaint (charWheelRedraw_ = !canSlide), so the slide is driven entirely through the cheap band-only pump path. - A dedicated, slightly longer ring-slide duration (kCharWheelRingSlideMs = 280 ms) so the lens spans enough frames to read as motion at this screen's frame rate. On hardware the slide now renders ~7-8 frames (~25 fps) instead of ~2. Add a native test that drawCharWheelBand draws the ring without the chips / label.

3 weeks ago

test_datetime_field

Add date-time field model and widget for button boards Introduce a field-by-field date-time editor for button-only boards: a pure, host-testable edit model (DtField, DateTimeFieldModel, daysInMonth, clampDay, dtIncrement/dtDecrement/dtNextField/dtAdvanceOrCommit) in the header, plus a solid-color renderer (drawDateTimeField) in the .cpp that draws YYYY-MM-DD HH:MM and a zone line, highlighting the active field. All ranges wrap; year/month changes re-clamp the day (leap-Feb aware). Adds native model tests. Claude-Session: https://claude.ai/code/session_01U878bNJtg9bCvAMM8CFKqS

1 week ago

test_dialog_component

feat(dialog): 4-zone layout — glyph / title / body / actions Restructure the alert/notice dialog into four stacked zones so the warning glyph sits in its own top row and the title owns the full card width, wrapping to a second HEADING row instead of ellipsizing. "Locked out" / "Last attempt" now read in full with the warning glyph centered above — fixing both the title truncation and the inline-glyph misalignment. The body word-wrap is factored into a shared wrapLines helper (appendWord gains a font parameter) reused by the title; the ellipsis-based fitTitle is removed. Native: 995/995 (dialog long-title test updated to the wrapping behavior). Builds: sticks3, m5core_ink (mono branch). Verified on StickS3 (lockout / wipe).

3 weeks ago

test_dropdown_component

feat(settings): wire Stepper to PIN-arc sweep + Dropdown to language Make two real StickS3 settings drive the new catalog components: - Stepper -> PIN timing-arc sweep duration. The arc sweep is now an owner-configurable, NVS-persisted setting (pin_arc_ms) with an enforced viable minimum: pin_arc gains runtime-duration overloads (constexpr default preserved), AppContext::pinArcSweepMs is loaded+clamped at boot, and PinEntryUI reads it via AuthEntryUI::setSweepDurationMs. A new SECURITY item opens a stepper edit overlay (BtnA nudge, BtnB tap=save/hold=cancel). - Dropdown -> language. The DEVICE > Language item opens a single-select popover listing every language with the applied one marked; committing calls i18n::setLang (already NVS-persisted). Replaces the in-place enum cycle. Adds SettingsItem::PinSweep + ItemAction::EditStepper/PickEnum, the SetPinSweep i18n string, and a RecordingCanvas dropdown test. settings_nav test updated for the new SECURITY count and the language/pin-sweep actions.

4 weeks ago

feat(ui): OKLab TOTP-countdown gradient + ease-in-out/back curves OKLab perceptual gradient (src/ui/core/gradient.{h,cpp}): sRGB->linear->OKLab lerp baked once into a small RGB565 LUT (33-entry TOTP countdown green->amber->red), no per-pixel cube-roots. Applied to the TOTP countdown bar via a new progress_component fillColor override (color panels only; mono/e-paper keep flat ink). Tween gains easeInOutQuad (smoother long slides) + easeOutBack (subtle spring) alongside easeOutQuad. Host tests for the LUT (endpoints/monotonic) + the new curves. Recovered from a rate-limited agent session (build + 904 native tests verified).

4 weeks ago

test_header_component

Show battery as a fixed-width gauge in the header The status cluster drew the battery as "NN%" text; a full "100%" is four characters and starved the title on the 135 px Stick header, forcing an early ellipsis. Replace it with a fixed-width gauge glyph (outline + terminal + proportional fill, tinted green while charging and red at <=15%). The level reads at a glance from the fill and the title gets the reclaimed width. Removes the now-unused percentage-formatting helper. Claude-Session: https://claude.ai/code/session_01SV58JXhfxhhc9DC6vdjBo4

4 days ago

test_hold_button

fix(test): configure clangd and clang-tidy for native test tree (#3) * fix(test): configure clangd and clang-tidy for native test tree - Generate compile_commands.json from pio run -e native -t compiledb so clangd resolves src/ headers in test files without errors. - Fix .clangd PathMatch regex (relative path, not absolute) and add -I.pio/build/native/unity_config so unity_config.h resolves. - Remove overly broad Remove: -I* that was stripping all added paths. - Add -Wno-format-security and UnusedIncludes: None to test block. - Add test/.clang-tidy that inherits the root config and: - Allows test_* functions to use snake_case (FunctionIgnoredRegexp) - Disables cert-err33-c, cert-dcl50-cpp, cppcoreguidelines-pro-type-cstyle-cast, bugprone-argument-comment, bugprone-misplaced-widening-cast, bugprone-implicit-widening-of-multiplication-result, misc-unused-using-decls — all false positives in test context. - Add /* intentionally empty */ comment to all empty setUp()/tearDown() bodies (28 files) to suppress SonarLint c:S1186. - Fix test_vault_task: change relative include ../../src/vault/vault_task.h to vault/vault_task.h (consistent with all other test files; -Isrc already in compile flags). - Reorganise test/ into a module-mirroring subtree layout: test/ble/, test/crypto/, test/hal/, test/ota/, test/platform/, test/states/, test/totp/, test/ui/, test/vault/, test/web/ (old flat directories removed). - Add sonar-project.properties exclusion e4: suppress c:S1186 on test/**/*.cpp at SonarCloud level. All 468 native unit tests pass (pio test -e native). * fix(clangd): add -DESP_PLATFORM to global CompileFlags All Kleidos targets are ESP32/ESP32-S3 (framework = espidf), so ESP_PLATFORM is always defined at compile time. Without this flag in the clangd config, the language server was activating #else branches inside #ifdef ESP_PLATFORM blocks, hiding device code and showing false diagnostics for src/ files. The test/** fragment already has Remove: -DESP_PLATFORM, so native unit-test files continue to compile without the flag. * revert(clangd): remove incorrect -DESP_PLATFORM from global CompileFlags Adding -DESP_PLATFORM globally caused 'too many errors' in src/ files because compile_commands.json was generated from the native env and lacked all ESP-IDF/Xtensa include paths. The correct fix is to generate compile_commands.json from a device env: ./scripts/generate_compiledb.sh sticks3 This populates the DB with proper -DESP_PLATFORM and all IDF includes for every src/ translation unit. The symlink at the project root then points clangd to the right compilation context.

1 month ago

test_icon_component

Reconcile the icon set and header brand mark to the v2 design Match the eight system icons to the approved handoff geometry (docs/design/handoff/assets/icons): Settings becomes three sliders with knobs (was a gear), and add Credential (ID card) and Totp (circular- arrow timer, distinct from the plain clock); Lock, Key, Battery, Ble and Warning are reshaped to the handoff strokes. Icons stay primitive-drawn so they render backend-uniform on color TFT and 1-bit e-paper at zero flash cost. Switch the header brand mark to the ring-less keyhole glyph (logo-glyph); the full ring mark stays for >=24 px branding (lock screen, onboarding, about). Add native tests for the icon catalog and the header mark. Claude-Session: https://claude.ai/code/session_01SV58JXhfxhhc9DC6vdjBo4

4 days ago

test_layout_context

Render CoreInk text with the real AA fonts via ordered dither The CoreInk e-paper drew UI text with the crude 5x7 bitmap font because the anti-aliased atlas path was gated off on monochrome panels. Enable it: CoreInk now uses the same Inter / JetBrains Mono atlases as the color fleet, mapping each glyph's coverage to ink/paper through an 8x8 Bayer ordered dither (with a small weight bias so hairlines survive on e-ink) instead of an RGB565 alpha blend. Sizes are raised to the e-ink legibility floor (body 16, title 24) via a per-device mono floor in the metrics resolver. The color alpha-blend path is unchanged and every native-scale color glyph stays byte-identical; the bitmap font remains compiled as a fallback. Claude-Session: https://claude.ai/code/session_01SV58JXhfxhhc9DC6vdjBo4

4 days ago

test_mask_glyph

feat(ui): bullet password mask + animated Stick tab indicator Replace the ASCII '*' password mask with U+2022 BULLET rendered in the active font (atlas-confirmed in every generated AA size). Add a shared, host-testable ui::mask_glyph helper that builds a bounded UTF-8 mask row (3 bytes per bullet, never splitting a multi-byte glyph, never emitting plaintext) and route the credential-detail field and the text-field component through it. Buffers are sized for the 3-byte glyph and the field tail trim now advances by whole code points. Masking behavior and the no-plaintext guarantee are unchanged; only the glyph differs. Redesign the Stick (2-button) page indicator: move it out of the brand/battery header into a top-edge segmented underline on the divider line (hairline kSubtle565 track inset by space.md, one segment per page, a bright kPrimary565 pill on the active segment). Slide the pill between segments on a page flip via ui::Tween (180 ms, Easing::InOutQuad), gated to Refresh::Instant; e-paper/mono snap with no animation. The slide is cooperative (one frame per tick while moving, static at rest) and reuses the existing menu-anim pump. Reserve a small band below the header so the list never collides with the pill. Geometry + timing live in a pure host-testable ui::tab_indicator header. The 3-button Gray tab path is untouched; the whole indicator stays gated behind !HAS_KEYBOARD. Add native tests for the mask-row builder and the indicator geometry, and update the text-field component test to expect the bullet.

4 weeks ago

test_menu_list_component

feat(menu-list): primary selection border + overlay-aware band clip Add a 2 px kPrimary left-edge border to the selected row on color panels so the focus reads at a glance beyond the (softer) accent band fill; it travels with both the static per-row fill and the animated sliding band. Mono panels keep their inverted strip + paper marker bar unchanged. Add an optional overlayReserve rect to MenuListProps: when a floating overlay (bottom toast / top banner) is drawn over the list, the selection band is clipped to the overlay edge so the highlight never bleeds under it, keeping exactly one fully on-screen focus highlight. Empty reserve (default) is a no-op, so all existing callers and tests are unchanged. Cover the border and the overlay clip with new component tests.

4 weeks ago

Make the PIN timing-arc fill animate smoothly again The dial filled in ~10 coarse blocks because each full repaint took ~300 ms, so the loop only sampled the continuous fill ~3 times/second. The cost was the AA ring rendering: isqrt64 ran a Newton-Raphson with a 64-bit divide per iteration (the ESP32-S3 has no hardware 64-bit divide) once per pixel, over the whole dial box, twice. The fill VALUE was always correct — only the redraw cadence was slow. - isqrt64: replace the division-based Newton form with a division-free binary (digit-by-digit) integer sqrt — only shifts/adds/compares, identical result. - fillAnnulusSweepAA / strokeCircleAA: gate the per-pixel sqrt behind a cheap d^2 band test, so the inner hole and box corners (most of a thin ring's box) are skipped without a sqrt; no change to the coverage math. - Add fillAnnulusGaugeAA: paint the dim track and the bright progress wedge in one band pass (distance/angle computed once), and use it in the PIN dial. Net: the dial repaint drops from ~300 ms to ~67 ms, so the arc now fills ~44 frames over a 3 s sweep (was ~10) — a smooth growing bar. Visually identical. Add a DEBUG_SERIAL_BUTTONS trace at the timing-arc output (elapsed/fill%/digit per rendered frame) for on-device QA, and native boundary tests asserting the exact digit slices (e.g. T=3000: 299 ms -> 0, 300 ms -> 1, and the 9 -> 0 wrap).

3 weeks ago

test_pin_dial_component

Refine the PIN dial: A-gauge ring, single moving marker, B-Clear footer Owner-approved dial refinement (#41). Replaces the busy original (0-9 ring + 10 ticks + double-circle outline all competing) with variant A's clean single-band gauge that collars the hero digit, and reintroduces orientation without the overload: - RING: one thick same-tone gauge (track + bright swept fill, dim outlines), not the thin double circle. - NUMERALS: 0-9 around the ring, SMALL, stratified by presence — active digit cyan-bold, the rest muted — so only one numeral claims attention. - MARKERS: a single highlight on the marker the sweep is currently at (the active digit), reverting to dim as it passes (new activeMarker prop) — not the cumulative 0..digit lit before. - FOOTER: BtnB physically clears, so the action bar reads 'A Hold / B Clear' (new PinClear i18n string); the misleading 'B SHAKE' and the redundant 'Shake to clear' gesture-hint line are gone (shake still clears, no on-screen hint needed now that B Clear is shown). Adds 3 pin_dial tests (transient marker: only-active, none-at-idle, follows the value). HW-verified on the StickS3 (MAC by id); text already renders through the AA atlases. Mockup: docs/design/mockups/pin-entry-aplus.svg.

3 weeks ago

test_popup_queue

Tidy popup queue params, toast array, and wrap loop (S107/S5945/S924/S886) Introduce a ModalLabels struct so beginModal/beginChoice drop under the 7-parameter limit; update the popup_system callers and native tests. Replace the C-style ToastSlot array with std::array. Flatten the word-wrap fill loop to a single break and move the line-count loop's text-end check into an explicit guard. Claude-Session: https://claude.ai/code/session_01Y7Cy1HetRp6TZAUAeekN8X

2 weeks ago

test_progress_component

feat(ui): add Phase 3 catalog components (MenuList/Dialog/Toast/Progress/Button/TextField/TabBar) Add seven stateless reference components to the in-house UI library, each props-in / draw-out over the ui::Canvas seam with a Mono1/e-paper branch and host RecordingCanvas tests across the four canonical geometries: - MenuList / ListRow — scrollable list (absorbs OnDeviceMenu): selection, scroll window, toggle/value/chevron trailers, scroll rail. - Dialog — centered modal/confirm card (title + wrapped body + 1-2 actions). - Toast / Banner — transient overlay + status strip (absorbs popup_system raster half + status_notifier). - ProgressBar / ProgressArc — linear bar + integer-math circular fill arc. - Button — pressable button with visual states (normal/focused/pressed/ disabled/danger). - TextField — single-line text/password field with caret + tail overflow. - TabBar — horizontal tab strip with a selected tab. Gap A: interactive components expose a visual state/selected/focused prop and render each state now; the focus model that drives selection lands later. Gap B: overflow/scroll/long-text handled by bounds math (visible-window + fit-to-rect) since Canvas::clip is advisory — no HW clip added. Additive only: new component sources + suites, plus platformio.ini build/test registrations and a docs/groups.dox group extension.

1 month ago

test_qr_component

Group drawQr dark/light colors into QrColors struct (S107) The explicit-color drawQr overload took eight parameters. Introduce a small ui::QrColors { dark, light } struct so the two RGB565 colors travel together, dropping the parameter count and reading more clearly at the call sites. Update the admin QR-card caller and the native qr_component test. Claude-Session: https://claude.ai/code/session_01Y7Cy1HetRp6TZAUAeekN8X

2 weeks ago

test_qr_encoder

feat(ui): add in-house QR encoder + scalable drawQr component Self-contained ISO/IEC 18004 byte-mode QR encoder (no vendoring, no deps): RS ECC over GF(256), block interleaving, function-pattern placement, the 8 mask patterns scored by the 4 penalty rules, and format/version info. Auto smallest version (1..10, documented RAM cap), ECC L/M/Q/H, malloc-free. Fix the data-placement bug from the partial work: the zigzag direction was a running toggle; the correct rule is per-column ((right & 2) == 0) XOR (col < 6), and the byte-aligned padding / two-copy format-info mapping were off. The encoder now matches a trusted generator (segno) module-for-module on pinned known-answer vectors (HELLO WORLD v1-M mask4, a URL v2-M mask2) and decodes across versions/ECC levels. Add ui::drawQr (scalable, quiet zone, theme tokens) + ui::qr::wifi escaped payload builder, and replace the admin WiFi-QR finder-pattern placeholder with a real scannable QR. Tests: encoder (known-answer + auto-version + caps), payload escaping, component rendering.

4 weeks ago

test_qr_payload

feat(ui): add in-house QR encoder + scalable drawQr component Self-contained ISO/IEC 18004 byte-mode QR encoder (no vendoring, no deps): RS ECC over GF(256), block interleaving, function-pattern placement, the 8 mask patterns scored by the 4 penalty rules, and format/version info. Auto smallest version (1..10, documented RAM cap), ECC L/M/Q/H, malloc-free. Fix the data-placement bug from the partial work: the zigzag direction was a running toggle; the correct rule is per-column ((right & 2) == 0) XOR (col < 6), and the byte-aligned padding / two-copy format-info mapping were off. The encoder now matches a trusted generator (segno) module-for-module on pinned known-answer vectors (HELLO WORLD v1-M mask4, a URL v2-M mask2) and decodes across versions/ECC levels. Add ui::drawQr (scalable, quiet zone, theme tokens) + ui::qr::wifi escaped payload builder, and replace the admin WiFi-QR finder-pattern placeholder with a real scannable QR. Tests: encoder (known-answer + auto-version + caps), payload escaping, component rendering.

4 weeks ago

test_screen_painter

Drive the Stick vault-menu animation through ScreenPainter The Stick menu's per-frame animation (drawStickMenuAnimFrame) cleared and recomposed the whole screen every tick — header, list, footer, tab pill — then relied on the single shadow-FB blit for flicker-freedom, so it re-blitted the full 135x240 frame ~8-10 times per scroll/highlight slide even though only the list body moves. Route it through ui::ScreenPainter instead: the header+tab band repaints only when the tab pill is sliding (a tab switch), the list body repaints every frame (clipped to its band, which also masks the sub-row overshoot the header/footer used to mask), and the footer is left untouched in the persistent shadow. A pure scroll now blits just the body band (~40% less per frame) and skips the header/footer re-render. Add a startFull=false ScreenPainter constructor so a short-lived per-frame painter driving already-painted chrome does not force a full repaint; host test covers it. Verified on StickS3 hardware: render is pixel-correct (no uncleared bands) and RENDERSTATS confirms the animation frames dropped below the full-screen blit. First screen migrated onto the central dirty-only mechanism; the menu's status-tick full redraw (battery/BLE) is the next target.

3 weeks ago

test_status_chip_component

feat(ui): add Phase 2 stateless reference components (ActionBar/Header/BrandMark/StatusChip) First catalog components on the Canvas + DeviceProfile foundation (plan/ui-library-modular-reimpl.md Phase 2). Stateless: props in -> draw via Canvas -> return, reading the orthogonal DeviceProfile for device adaptation. Purely additive — no existing screen/state/widget/ Canvas changed. - ActionBar (action_bar_component): affordance-driven footer hints. The same ActionBarProps render the device-correct presentation picked from the profile capabilities — 2-button A/B, 3-button A/B/C, CoreInk dial ^/O/v, keyboard key hints, touch bare labels. Mono1/e-paper path uses a top divider rule + cell inversion instead of the SURFACE fill + ACCENT hold-fill. - Header (header_component): title + subtitle + right-packed status cluster (battery %, BLE, WiFi, lock). Density/orientation-aware via LayoutContext; Mono1 uses a bottom divider rule, ink-only tokens. - BrandMark (brand_mark_component): credential brand badge. Reuses the brands::findBrand/drawBrand registry+atlas on device; falls back to a deterministic per-domain letter tile (host-testable). Mono1 outlines. - StatusChip (status_chip_component): self-sizing status pill using radius.pill (clamped), token color per ChipStyle, knockout/outline text. Tests (host, RecordingCanvas): 30 new cases across 4 suites, incl. the per-affordance ActionBar assertions (2btn/3btn/dial/keyboard/touch), the Mono1 branches, brand letter-fallback + deterministic color, and chip sizing/geometry. Shared per-variant DeviceProfile factories in test/support/device_profiles.h. Verified: fleet 5/5 -Werror (sticks3/gray/cardputer/tdeck/m5core_ink), native 720/720 (+30), pio check 3 pre-existing HIGH only, all guards + pre-commit pass.

1 month ago

test_status_notifier

fix(test): configure clangd and clang-tidy for native test tree (#3) * fix(test): configure clangd and clang-tidy for native test tree - Generate compile_commands.json from pio run -e native -t compiledb so clangd resolves src/ headers in test files without errors. - Fix .clangd PathMatch regex (relative path, not absolute) and add -I.pio/build/native/unity_config so unity_config.h resolves. - Remove overly broad Remove: -I* that was stripping all added paths. - Add -Wno-format-security and UnusedIncludes: None to test block. - Add test/.clang-tidy that inherits the root config and: - Allows test_* functions to use snake_case (FunctionIgnoredRegexp) - Disables cert-err33-c, cert-dcl50-cpp, cppcoreguidelines-pro-type-cstyle-cast, bugprone-argument-comment, bugprone-misplaced-widening-cast, bugprone-implicit-widening-of-multiplication-result, misc-unused-using-decls — all false positives in test context. - Add /* intentionally empty */ comment to all empty setUp()/tearDown() bodies (28 files) to suppress SonarLint c:S1186. - Fix test_vault_task: change relative include ../../src/vault/vault_task.h to vault/vault_task.h (consistent with all other test files; -Isrc already in compile flags). - Reorganise test/ into a module-mirroring subtree layout: test/ble/, test/crypto/, test/hal/, test/ota/, test/platform/, test/states/, test/totp/, test/ui/, test/vault/, test/web/ (old flat directories removed). - Add sonar-project.properties exclusion e4: suppress c:S1186 on test/**/*.cpp at SonarCloud level. All 468 native unit tests pass (pio test -e native). * fix(clangd): add -DESP_PLATFORM to global CompileFlags All Kleidos targets are ESP32/ESP32-S3 (framework = espidf), so ESP_PLATFORM is always defined at compile time. Without this flag in the clangd config, the language server was activating #else branches inside #ifdef ESP_PLATFORM blocks, hiding device code and showing false diagnostics for src/ files. The test/** fragment already has Remove: -DESP_PLATFORM, so native unit-test files continue to compile without the flag. * revert(clangd): remove incorrect -DESP_PLATFORM from global CompileFlags Adding -DESP_PLATFORM globally caused 'too many errors' in src/ files because compile_commands.json was generated from the native env and lacked all ESP-IDF/Xtensa include paths. The correct fix is to generate compile_commands.json from a device env: ./scripts/generate_compiledb.sh sticks3 This populates the DB with proper -DESP_PLATFORM and all IDF includes for every src/ translation unit. The symlink at the project root then points clangd to the right compilation context.

1 month ago

test_stepper_component

feat(ui): add reusable numeric Stepper component A stateless ui::Canvas component rendering a '- value +' numeric control with min/max/step clamping and saturating value helpers (stepperClamp/ stepperNext/stepperPrev). The focused affordance highlights with the accent chip on color panels and inverts to ink on Mono1/e-paper. Adds a RecordingCanvas host test covering the value math, the two-chip render, the centered value, the empty-rect guard, and the mono branch.

4 weeks ago

test_tab_bar_component

feat(ui): add Phase 3 catalog components (MenuList/Dialog/Toast/Progress/Button/TextField/TabBar) Add seven stateless reference components to the in-house UI library, each props-in / draw-out over the ui::Canvas seam with a Mono1/e-paper branch and host RecordingCanvas tests across the four canonical geometries: - MenuList / ListRow — scrollable list (absorbs OnDeviceMenu): selection, scroll window, toggle/value/chevron trailers, scroll rail. - Dialog — centered modal/confirm card (title + wrapped body + 1-2 actions). - Toast / Banner — transient overlay + status strip (absorbs popup_system raster half + status_notifier). - ProgressBar / ProgressArc — linear bar + integer-math circular fill arc. - Button — pressable button with visual states (normal/focused/pressed/ disabled/danger). - TextField — single-line text/password field with caret + tail overflow. - TabBar — horizontal tab strip with a selected tab. Gap A: interactive components expose a visual state/selected/focused prop and render each state now; the focus model that drives selection lands later. Gap B: overflow/scroll/long-text handled by bounds math (visible-window + fit-to-rect) since Canvas::clip is advisory — no HW clip added. Additive only: new component sources + suites, plus platformio.ini build/test registrations and a docs/groups.dox group extension.

1 month ago

test_tab_indicator

fix(ui): ellipsis truncation, safe-area margins, stronger tab indicator - List rows and header titles now ellipsis-truncate (U+2026) instead of hard-clipping at the panel edge, so a cut label reads as cut (typography-small-displays §Truncation). Header titles are width-clamped to the band before the status cluster so a long host never touches the right edge. - Action-bar cells inset by the safe-area side margin so an edge verb (e.g. left OPEN VAULT) never touches the bezel / rounded-corner clip. - Tab indicator: a full-width subtle per-segment track + bright active pill, bumped to 4 px (>= 3:1 non-text visibility) with a gap above the list, so it reads as a 1-of-N pager rather than a 1 px title underline. Keeps the 180 ms slide; Gray 3-button path untouched.

4 weeks ago

test_text_box_component

Test the software clip and drawTextBox Add host unit coverage for the new clip feature: - test_clip_window: the integer rect intersection and ClipWindow (full / partial / disjoint overlap, inactive pass-through, per-pixel membership). - test_display_graphics: a ClipSurface fake that honors a ClipWindow exactly like the drivers, asserting an armed clip drops every fill / single-pixel / text-glyph pixel outside the window (paintedOutside == 0) and that clearing the clip restores full-surface drawing. - test_text_box_component: drawTextBox wraps long text into multiple rows, arms one clip equal to the rect (then clipReset), draws every row under the active clip, cuts vertical overflow at the rect bottom, and the no-wrap / empty-input guards.

3 weeks ago

test_text_field_component

feat(ui): bullet password mask + animated Stick tab indicator Replace the ASCII '*' password mask with U+2022 BULLET rendered in the active font (atlas-confirmed in every generated AA size). Add a shared, host-testable ui::mask_glyph helper that builds a bounded UTF-8 mask row (3 bytes per bullet, never splitting a multi-byte glyph, never emitting plaintext) and route the credential-detail field and the text-field component through it. Buffers are sized for the 3-byte glyph and the field tail trim now advances by whole code points. Masking behavior and the no-plaintext guarantee are unchanged; only the glyph differs. Redesign the Stick (2-button) page indicator: move it out of the brand/battery header into a top-edge segmented underline on the divider line (hairline kSubtle565 track inset by space.md, one segment per page, a bright kPrimary565 pill on the active segment). Slide the pill between segments on a page flip via ui::Tween (180 ms, Easing::InOutQuad), gated to Refresh::Instant; e-paper/mono snap with no animation. The slide is cooperative (one frame per tick while moving, static at rest) and reuses the existing menu-anim pump. Reserve a small band below the header so the list never collides with the pill. Geometry + timing live in a pure host-testable ui::tab_indicator header. The 3-button Gray tab path is untouched; the whole indicator stays gated behind !HAS_KEYBOARD. Add native tests for the mask-row builder and the indicator geometry, and update the text-field component test to expect the bullet.

4 weeks ago

test_toast_component

Draw the toast severity marker on color panels too (C-5) Color-panel toasts conveyed severity by the left accent-bar hue alone: the title was white+bold for every severity, so a deuteranope saw an identical info and warning toast (the marker glyph was mono-only). Draw the per-severity marker (i / + / ! / x / *) before the title on every panel — in the accent hue on color, in ink on Mono1 — giving severity a shape second channel (1.4.1). Reserve the marker advance in the card width math so the title never clips. Add a native test asserting two severities draw distinct markers on a color profile. HW-verified on StickS3: info shows a cyan 'i', success a green '+', warning a red 'x'.

3 weeks ago