From aad8b6adce4da1bd5ff32841044693de847765f8 Mon Sep 17 00:00:00 2001 From: Jason P Date: Wed, 14 Jan 2026 10:48:12 -0600 Subject: [PATCH] Testing unveiled some issues - fixed with these changes --- src/graphics/draw/MenuHandler.cpp | 1 + src/graphics/draw/NodeListRenderer.cpp | 6 ++---- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/src/graphics/draw/MenuHandler.cpp b/src/graphics/draw/MenuHandler.cpp index 1b3639dca..13e7d0dd2 100644 --- a/src/graphics/draw/MenuHandler.cpp +++ b/src/graphics/draw/MenuHandler.cpp @@ -1430,6 +1430,7 @@ void menuHandler::nodeNameLengthMenu() } config.display.use_long_node_name = option.value; + saveUIConfig(); LOG_INFO("Setting names to %s", option.value ? "long" : "short"); }); diff --git a/src/graphics/draw/NodeListRenderer.cpp b/src/graphics/draw/NodeListRenderer.cpp index 4befa4320..9d6780130 100644 --- a/src/graphics/draw/NodeListRenderer.cpp +++ b/src/graphics/draw/NodeListRenderer.cpp @@ -176,6 +176,7 @@ int calculateMaxScroll(int totalEntries, int visibleRows) void drawColumnSeparator(OLEDDisplay *display, int16_t x, int16_t yStart, int16_t yEnd) { + x = (currentResolution == ScreenResolution::High) ? x - 2 : (currentResolution == ScreenResolution::Low) ? x - 1 : x; for (int y = yStart; y <= yEnd; y += 2) { display->setPixel(x, y); } @@ -599,11 +600,8 @@ void drawNodeListScreen(OLEDDisplay *display, OLEDDisplayUiState *state, int16_t // Draw column separator if (currentResolution != ScreenResolution::UltraLow && shownCount > 0) { const int firstNodeY = y + 3; - const int tempColumnWidth = (currentResolution == ScreenResolution::High) ? columnWidth - 2 - : (currentResolution == ScreenResolution::Low) ? columnWidth - 1 - : columnWidth; for (int horizontal_offset = 1; horizontal_offset < totalColumns; horizontal_offset++) { - drawColumnSeparator(display, tempColumnWidth * horizontal_offset, firstNodeY, lastNodeY); + drawColumnSeparator(display, columnWidth * horizontal_offset, firstNodeY, lastNodeY); } }