diff options
author | Aidan MacDonald <amachronic@protonmail.com> | 2022-09-19 11:10:54 +0100 |
---|---|---|
committer | Aidan MacDonald <amachronic@protonmail.com> | 2022-10-02 09:21:25 -0400 |
commit | 9ce5b2a2eda5c7101898476adaa3533e0ba5db82 (patch) | |
tree | facbfe9721d907b1a92f2b113b262374c6e2917e /apps/gui/list.h | |
parent | c1d75a6bcb30260eba288f643261c4a57959f45f (diff) | |
download | rockbox-9ce5b2a2eda5c7101898476adaa3533e0ba5db82.tar.gz rockbox-9ce5b2a2eda5c7101898476adaa3533e0ba5db82.zip |
gui: Remove show/hide selection option in lists
The implementation of the "show_selection_marker" option in
lists isn't great. It's a cosmetic option used to hide the
selection, but it causes the list to do funny things to the
selected_item and doesn't play nice with voiced menus, since
these rely on the selection to determine what item is spoken.
There are only two user-facing lists that use the option, the
"Rockbox Info" screen and a menu in the superdom plugin. The
rest are debug screens, and cosmetics don't matter much there.
Given how little used the option is, and its issues, removing
it seems reasonable.
Change-Id: I2c70b3e4c74ff3cc6dbac46366a371d271dd2d58
Diffstat (limited to 'apps/gui/list.h')
-rw-r--r-- | apps/gui/list.h | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/apps/gui/list.h b/apps/gui/list.h index 2df33b7541..388e3d2006 100644 --- a/apps/gui/list.h +++ b/apps/gui/list.h | |||
@@ -161,7 +161,6 @@ struct gui_synclist | |||
161 | /* whether the text of the whole items of the list have to be | 161 | /* whether the text of the whole items of the list have to be |
162 | * scrolled or only for the selected item */ | 162 | * scrolled or only for the selected item */ |
163 | bool scroll_all; | 163 | bool scroll_all; |
164 | bool show_selection_marker; /* set to true by default */ | ||
165 | int nb_items; | 164 | int nb_items; |
166 | int selected_item; | 165 | int selected_item; |
167 | 166 | ||
@@ -235,8 +234,6 @@ extern void gui_synclist_del_item(struct gui_synclist * lists); | |||
235 | extern void gui_synclist_limit_scroll(struct gui_synclist * lists, bool scroll); | 234 | extern void gui_synclist_limit_scroll(struct gui_synclist * lists, bool scroll); |
236 | extern void gui_synclist_set_title(struct gui_synclist * lists, const char * title, | 235 | extern void gui_synclist_set_title(struct gui_synclist * lists, const char * title, |
237 | enum themable_icons icon); | 236 | enum themable_icons icon); |
238 | extern void gui_synclist_hide_selection_marker(struct gui_synclist *lists, | ||
239 | bool hide); | ||
240 | extern bool gui_synclist_item_is_onscreen(struct gui_synclist *lists, | 237 | extern bool gui_synclist_item_is_onscreen(struct gui_synclist *lists, |
241 | enum screen_type screen, int item); | 238 | enum screen_type screen, int item); |
242 | 239 | ||
@@ -304,7 +301,6 @@ struct simplelist_info { | |||
304 | const char *title; /* title to show on the list */ | 301 | const char *title; /* title to show on the list */ |
305 | int count; /* number of items in the list, each item is selection_size high */ | 302 | int count; /* number of items in the list, each item is selection_size high */ |
306 | int selection_size; /* list selection size, usually 1 */ | 303 | int selection_size; /* list selection size, usually 1 */ |
307 | bool hide_selection; | ||
308 | bool scroll_all; | 304 | bool scroll_all; |
309 | bool hide_theme; | 305 | bool hide_theme; |
310 | bool speak_onshow; /* list speaks first item or 'empty list' */ | 306 | bool speak_onshow; /* list speaks first item or 'empty list' */ |
@@ -350,7 +346,6 @@ void simplelist_addline(const char *fmt, ...); | |||
350 | /* setup the info struct. members not setup in this function need to be assigned manually | 346 | /* setup the info struct. members not setup in this function need to be assigned manually |
351 | members set in this function: | 347 | members set in this function: |
352 | info.selection_size = 1; | 348 | info.selection_size = 1; |
353 | info.hide_selection = false; | ||
354 | info.scroll_all = false; | 349 | info.scroll_all = false; |
355 | info.hide_theme = false; | 350 | info.hide_theme = false; |
356 | info.speak_onshow = true; | 351 | info.speak_onshow = true; |