From a72fbdda62af1bd468c9209e6fe76f12db3b2ed2 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 15 Jan 2026 19:22:40 +0000 Subject: [PATCH] Use consistent invertText variable for all text rendering Co-authored-by: lukestein <44452336+lukestein@users.noreply.github.com> --- src/activities/home/HomeActivity.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/activities/home/HomeActivity.cpp b/src/activities/home/HomeActivity.cpp index 92eff6b3..1eee0436 100644 --- a/src/activities/home/HomeActivity.cpp +++ b/src/activities/home/HomeActivity.cpp @@ -325,6 +325,9 @@ void HomeActivity::render() { } if (hasContinueReading) { + // When cover is shown, invert text colors on selection; without cover, invert normally + const bool invertText = coverRendered ? bookSelected : !bookSelected; + // Split into words (avoid stringstream to keep this light on the MCU) std::vector words; words.reserve(8); @@ -444,9 +447,6 @@ void HomeActivity::render() { renderer.drawRect(boxX, boxY, boxWidth, boxHeight, !bookSelected); } - // When cover is shown, invert text colors on selection; without cover, invert normally - const bool invertText = coverRendered ? bookSelected : !bookSelected; - for (const auto& line : lines) { renderer.drawCenteredText(UI_12_FONT_ID, titleYStart, line.c_str(), invertText); titleYStart += renderer.getLineHeight(UI_12_FONT_ID); @@ -485,9 +485,9 @@ void HomeActivity::render() { const int continueBoxY = continueY - continuePadding / 2; renderer.fillRect(continueBoxX, continueBoxY, continueBoxWidth, continueBoxHeight, bookSelected); renderer.drawRect(continueBoxX, continueBoxY, continueBoxWidth, continueBoxHeight, !bookSelected); - renderer.drawCenteredText(UI_10_FONT_ID, continueY, continueText, bookSelected); + renderer.drawCenteredText(UI_10_FONT_ID, continueY, continueText, invertText); } else { - renderer.drawCenteredText(UI_10_FONT_ID, continueY, "Continue Reading", !bookSelected); + renderer.drawCenteredText(UI_10_FONT_ID, continueY, "Continue Reading", invertText); } } else { // No book to continue reading