summaryrefslogtreecommitdiff
path: root/apps/plugins/lrcplayer.c
diff options
context:
space:
mode:
authorAidan MacDonald <amachronic@protonmail.com>2022-09-19 11:10:54 +0100
committerAidan MacDonald <amachronic@protonmail.com>2022-10-02 09:21:25 -0400
commit9ce5b2a2eda5c7101898476adaa3533e0ba5db82 (patch)
treefacbfe9721d907b1a92f2b113b262374c6e2917e /apps/plugins/lrcplayer.c
parentc1d75a6bcb30260eba288f643261c4a57959f45f (diff)
downloadrockbox-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/plugins/lrcplayer.c')
-rw-r--r--apps/plugins/lrcplayer.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/apps/plugins/lrcplayer.c b/apps/plugins/lrcplayer.c
index f42b96b5b3..6c26db7a33 100644
--- a/apps/plugins/lrcplayer.c
+++ b/apps/plugins/lrcplayer.c
@@ -2437,7 +2437,6 @@ static bool lrc_debug_menu(void)
2437{ 2437{
2438 struct simplelist_info info; 2438 struct simplelist_info info;
2439 rb->simplelist_info_init(&info, "Debug Menu", 6, NULL); 2439 rb->simplelist_info_init(&info, "Debug Menu", 6, NULL);
2440 info.hide_selection = true;
2441 info.scroll_all = true; 2440 info.scroll_all = true;
2442 info.get_name = lrc_debug_data; 2441 info.get_name = lrc_debug_data;
2443 return rb->simplelist_show_list(&info); 2442 return rb->simplelist_show_list(&info);