dandri/wadamesh
1
0
Fork 0
mirror of https://github.com/ALLFATHER-BV/wadamesh.git synced 2026-10-15 06:09:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(ui): Map side buttons show which one is selected (#576)

Browse Source 
The keyboard/trackball cursor is a 3 px accent outline, and on an accent
fill it disappears. setSelectionGlow already swaps to the contrast
outline (white, black on Day) there, but only for a fully opaque fill;
the Map's side buttons are accent at 70% over the tiles, so they kept
the accent outline and the selected one barely stood out. Any fill that
is at least half accent now takes the contrast outline.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Michael A. Cojocari
2026-09-29 20:20:18 -04:00
co-authored by Claude Opus 5.5
parent 500b24e38c
commit d6bc169017
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ui-touch/UITask.cpp
+5 -1
View File
@@ -1977,7 +1977,11 @@ static void setSelectionGlow(lv_obj_t* obj, bool selected, lv_style_selector_t s
lv_obj_remove_style(obj, &s_selection_glow_style, cursor_selector);
lv_obj_remove_style(obj, &s_selection_glow_contrast_style, cursor_selector);
if (selected) {
const bool on_accent = lv_obj_get_style_bg_opa(obj, cursor_part) == LV_OPA_COVER &&
// An accent outline vanishes on an accent fill, and not only a solid one:
// the Map's side buttons are accent at 70% over the tiles, and their cursor
// could barely be told from its neighbours (#576). Anything mostly accent
// takes the contrast outline instead.
const bool on_accent = lv_obj_get_style_bg_opa(obj, cursor_part) >= LV_OPA_50 &&
lv_color_to32(lv_obj_get_style_bg_color(obj, cursor_part)) ==
lv_color_to32(lv_color_hex(COLOR_ACCENT));
lv_obj_add_style(obj, on_accent ? &s_selection_glow_contrast_style