From 8a9fb33a6a2137beb7cce1a02cabbdd73e9ee78b Mon Sep 17 00:00:00 2001 From: Jakub <106778416+MarekZegare4@users.noreply.github.com> Date: Mon, 6 Jul 2026 20:39:11 +0200 Subject: [PATCH] fix(ui): drop {loc}/{time} placeholders in the room password keyboard MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The shared keyboard seeds {loc}/{time} on begin() for message composition, but a room/repeater login password is not a message — a placeholder token there is nonsensical and a footgun (picking one inserts literal "{time}" into the password). Clear them right after opening the keyboard in both room-login paths (context-menu Login… and the Enter-on-room prompt), matching the same fix already applied to preset names and waypoint labels. Builds green: WioTrackerL1_companion_solo_dual. Co-Authored-By: Claude Opus 4.8 --- examples/companion_radio/ui-new/QuickMsgScreen.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/examples/companion_radio/ui-new/QuickMsgScreen.h b/examples/companion_radio/ui-new/QuickMsgScreen.h index 0aeb180c..56461c5f 100644 --- a/examples/companion_radio/ui-new/QuickMsgScreen.h +++ b/examples/companion_radio/ui-new/QuickMsgScreen.h @@ -1228,6 +1228,7 @@ public: if (the_mesh.getContactByIdx(_sorted[_contact_sel], _sel_contact)) { _login_mode = true; _kb->begin("", 15); // room/repeater password: max 15 chars + _kb->clearPlaceholders(); // {loc}/{time} are for messages, not a password _phase = KEYBOARD; } } @@ -1345,6 +1346,7 @@ public: } else { _login_mode = true; _kb->begin("", 15); // room/repeater password: max 15 chars + _kb->clearPlaceholders(); // {loc}/{time} are for messages, not a password _phase = KEYBOARD; } return true;