From 358c6056efab45e31e6178336309fa19d33e4486 Mon Sep 17 00:00:00 2001 From: Christian Soffke Date: Wed, 27 May 2026 19:07:42 +0200 Subject: [PATCH] gui: skinned list: always set cfg to NULL when toggling theme Takes care of disabling scroll bars when the current activity doesn't change, but the theme is toggled. Applies to confirmation prompts, for example. Since we're updating the skin immediately afterwards, the cfg will be reset as needed. Change-Id: I8a62b6b4ad0abf5176ab6b3e39281021108cb59f --- apps/gui/viewport.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/gui/viewport.c b/apps/gui/viewport.c index 0c52b7afe3..5fcc2a17a4 100644 --- a/apps/gui/viewport.c +++ b/apps/gui/viewport.c @@ -113,6 +113,7 @@ static void toggle_theme(enum screen_type screen, bool force) if (!sb_get_persistent_title(i)) sb_set_title_text(NULL, Icon_NOICON, i); } + skinlist_set_cfg(screen, NULL); toggle_events(enable_event); if (is_theme_enabled(screen)) @@ -163,7 +164,6 @@ static void toggle_theme(enum screen_type screen, bool force) screens[screen].backdrop_show(NULL); #endif screens[screen].scroll_stop(); - skinlist_set_cfg(screen, NULL); skin_is_dirty(screen); /* remove dirty flag */ } /* let list initialize viewport in case viewport dimensions is changed. */