From 99cbe517978893374eccfa555f0b208efdb8b728 Mon Sep 17 00:00:00 2001 From: Jakub <106778416+MarekZegare4@users.noreply.github.com> Date: Thu, 18 Jun 2026 18:06:59 +0200 Subject: [PATCH] fix(ui): popup selection bar stops before the scroll-indicator gutter Matches every other list (drawList's row width - reserve): the highlight shouldn't paint over the indicator's column, just frame it. Co-Authored-By: Claude Sonnet 4.6 --- examples/companion_radio/ui-new/PopupMenu.h | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/examples/companion_radio/ui-new/PopupMenu.h b/examples/companion_radio/ui-new/PopupMenu.h index 4cfc00b4..fa2f7b50 100644 --- a/examples/companion_radio/ui-new/PopupMenu.h +++ b/examples/companion_radio/ui-new/PopupMenu.h @@ -106,7 +106,10 @@ struct PopupMenu { int py = list_y + i * item_h + 1; if (idx == _sel) { display.setColor(DisplayDriver::LIGHT); - display.fillRect(bx + 1, py - 1, bw - 2, item_h); + // Stops short of the scroll-indicator gutter, like every other list's + // selection bar (e.g. drawList's row width - reserve) — otherwise the + // bar paints over the indicator's column instead of framing it. + display.fillRect(bx + 1, py - 1, bw - 2 - arrow_w, item_h); display.setColor(DisplayDriver::DARK); } else { display.setColor(DisplayDriver::LIGHT);