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/debug_menu.c | |
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/debug_menu.c')
-rw-r--r-- | apps/debug_menu.c | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/apps/debug_menu.c b/apps/debug_menu.c index 1abeac2622..b034f25df0 100644 --- a/apps/debug_menu.c +++ b/apps/debug_menu.c | |||
@@ -227,7 +227,6 @@ static bool dbg_os(void) | |||
227 | 227 | ||
228 | simplelist_info_init(&info, IF_COP("Core and ") "Stack usage:", | 228 | simplelist_info_init(&info, IF_COP("Core and ") "Stack usage:", |
229 | MAXTHREADS IF_COP( + NUM_CORES ), &xoffset); | 229 | MAXTHREADS IF_COP( + NUM_CORES ), &xoffset); |
230 | info.hide_selection = true; | ||
231 | info.scroll_all = false; | 230 | info.scroll_all = false; |
232 | info.action_callback = dbg_threads_action_callback; | 231 | info.action_callback = dbg_threads_action_callback; |
233 | info.get_name = threads_getname; | 232 | info.get_name = threads_getname; |
@@ -341,7 +340,6 @@ static bool dbg_cpuinfo(void) | |||
341 | info.get_name = get_cpuinfo; | 340 | info.get_name = get_cpuinfo; |
342 | info.action_callback = cpuinfo_cb; | 341 | info.action_callback = cpuinfo_cb; |
343 | info.timeout = HZ; | 342 | info.timeout = HZ; |
344 | info.hide_selection = true; | ||
345 | info.scroll_all = true; | 343 | info.scroll_all = true; |
346 | return simplelist_show_list(&info); | 344 | return simplelist_show_list(&info); |
347 | } | 345 | } |
@@ -568,7 +566,6 @@ static bool dbg_partitions(void) | |||
568 | struct simplelist_info info; | 566 | struct simplelist_info info; |
569 | simplelist_info_init(&info, "Partition Info", NUM_DRIVES * 4, NULL); | 567 | simplelist_info_init(&info, "Partition Info", NUM_DRIVES * 4, NULL); |
570 | info.selection_size = 2; | 568 | info.selection_size = 2; |
571 | info.hide_selection = true; | ||
572 | info.scroll_all = true; | 569 | info.scroll_all = true; |
573 | info.get_name = dbg_partitions_getname; | 570 | info.get_name = dbg_partitions_getname; |
574 | return simplelist_show_list(&info); | 571 | return simplelist_show_list(&info); |
@@ -1722,7 +1719,6 @@ static bool dbg_ata_smart(void) | |||
1722 | struct simplelist_info info; | 1719 | struct simplelist_info info; |
1723 | simplelist_info_init(&info, "S.M.A.R.T. Data [CONTEXT to dump]", 1, NULL); | 1720 | simplelist_info_init(&info, "S.M.A.R.T. Data [CONTEXT to dump]", 1, NULL); |
1724 | info.action_callback = ata_smart_callback; | 1721 | info.action_callback = ata_smart_callback; |
1725 | info.hide_selection = true; | ||
1726 | info.scroll_all = true; | 1722 | info.scroll_all = true; |
1727 | return simplelist_show_list(&info); | 1723 | return simplelist_show_list(&info); |
1728 | } | 1724 | } |
@@ -1776,7 +1772,6 @@ static bool dbg_disk_info(void) | |||
1776 | info.title = title; | 1772 | info.title = title; |
1777 | #endif | 1773 | #endif |
1778 | info.action_callback = disk_callback; | 1774 | info.action_callback = disk_callback; |
1779 | info.hide_selection = true; | ||
1780 | info.scroll_all = true; | 1775 | info.scroll_all = true; |
1781 | return simplelist_show_list(&info); | 1776 | return simplelist_show_list(&info); |
1782 | } | 1777 | } |
@@ -1845,7 +1840,6 @@ static bool dbg_dircache_info(void) | |||
1845 | int syncbuild = 0; | 1840 | int syncbuild = 0; |
1846 | simplelist_info_init(&info, "Dircache Info", 8, &syncbuild); | 1841 | simplelist_info_init(&info, "Dircache Info", 8, &syncbuild); |
1847 | info.action_callback = dircache_callback; | 1842 | info.action_callback = dircache_callback; |
1848 | info.hide_selection = true; | ||
1849 | info.scroll_all = true; | 1843 | info.scroll_all = true; |
1850 | return simplelist_show_list(&info); | 1844 | return simplelist_show_list(&info); |
1851 | } | 1845 | } |
@@ -1903,7 +1897,6 @@ static bool dbg_tagcache_info(void) | |||
1903 | struct simplelist_info info; | 1897 | struct simplelist_info info; |
1904 | simplelist_info_init(&info, "Database Info", 8, NULL); | 1898 | simplelist_info_init(&info, "Database Info", 8, NULL); |
1905 | info.action_callback = database_callback; | 1899 | info.action_callback = database_callback; |
1906 | info.hide_selection = true; | ||
1907 | info.scroll_all = true; | 1900 | info.scroll_all = true; |
1908 | 1901 | ||
1909 | /* Don't do nonblock here, must give enough processing time | 1902 | /* Don't do nonblock here, must give enough processing time |
@@ -2176,7 +2169,6 @@ static bool dbg_fm_radio(void) | |||
2176 | radio_hardware_present() ? "yes" : "no"); | 2169 | radio_hardware_present() ? "yes" : "no"); |
2177 | 2170 | ||
2178 | info.action_callback = radio_hardware_present()?radio_callback : NULL; | 2171 | info.action_callback = radio_hardware_present()?radio_callback : NULL; |
2179 | info.hide_selection = true; | ||
2180 | return simplelist_show_list(&info); | 2172 | return simplelist_show_list(&info); |
2181 | } | 2173 | } |
2182 | #endif /* CONFIG_TUNER */ | 2174 | #endif /* CONFIG_TUNER */ |
@@ -2448,7 +2440,6 @@ static bool dbg_talk(void) | |||
2448 | else | 2440 | else |
2449 | simplelist_info_init(&list, "Voice Information:", 2, &data); | 2441 | simplelist_info_init(&list, "Voice Information:", 2, &data); |
2450 | list.scroll_all = true; | 2442 | list.scroll_all = true; |
2451 | list.hide_selection = true; | ||
2452 | list.timeout = HZ; | 2443 | list.timeout = HZ; |
2453 | list.get_name = dbg_talk_get_name; | 2444 | list.get_name = dbg_talk_get_name; |
2454 | 2445 | ||
@@ -2488,7 +2479,6 @@ static bool dbg_isp1583(void) | |||
2488 | isp1583.scroll_all = true; | 2479 | isp1583.scroll_all = true; |
2489 | simplelist_info_init(&isp1583, "ISP1583", dbg_usb_num_items(), NULL); | 2480 | simplelist_info_init(&isp1583, "ISP1583", dbg_usb_num_items(), NULL); |
2490 | isp1583.timeout = HZ/100; | 2481 | isp1583.timeout = HZ/100; |
2491 | isp1583.hide_selection = true; | ||
2492 | isp1583.get_name = dbg_usb_item; | 2482 | isp1583.get_name = dbg_usb_item; |
2493 | isp1583.action_callback = isp1583_action_callback; | 2483 | isp1583.action_callback = isp1583_action_callback; |
2494 | return simplelist_show_list(&isp1583); | 2484 | return simplelist_show_list(&isp1583); |
@@ -2514,7 +2504,6 @@ static bool dbg_pic(void) | |||
2514 | pic.scroll_all = true; | 2504 | pic.scroll_all = true; |
2515 | simplelist_info_init(&pic, "PIC", pic_dbg_num_items(), NULL); | 2505 | simplelist_info_init(&pic, "PIC", pic_dbg_num_items(), NULL); |
2516 | pic.timeout = HZ/100; | 2506 | pic.timeout = HZ/100; |
2517 | pic.hide_selection = true; | ||
2518 | pic.get_name = pic_dbg_item; | 2507 | pic.get_name = pic_dbg_item; |
2519 | pic.action_callback = pic_action_callback; | 2508 | pic.action_callback = pic_action_callback; |
2520 | return simplelist_show_list(&pic); | 2509 | return simplelist_show_list(&pic); |
@@ -2533,7 +2522,6 @@ static bool dbg_skin_engine(void) | |||
2533 | #endif | 2522 | #endif |
2534 | simplelist_info_init(&info, "Skin engine usage", 0, NULL); | 2523 | simplelist_info_init(&info, "Skin engine usage", 0, NULL); |
2535 | simplelist_set_line_count(0); | 2524 | simplelist_set_line_count(0); |
2536 | info.hide_selection = true; | ||
2537 | FOR_NB_SCREENS(j) { | 2525 | FOR_NB_SCREENS(j) { |
2538 | #if NB_SCREENS > 1 | 2526 | #if NB_SCREENS > 1 |
2539 | simplelist_addline("%s display:", | 2527 | simplelist_addline("%s display:", |
@@ -2612,7 +2600,6 @@ static bool dbg_boot_data(void) | |||
2612 | boot_data.payload[i+1], boot_data.payload[i+2], boot_data.payload[i+3]); | 2600 | boot_data.payload[i+1], boot_data.payload[i+2], boot_data.payload[i+3]); |
2613 | } | 2601 | } |
2614 | 2602 | ||
2615 | info.hide_selection = true; | ||
2616 | return simplelist_show_list(&info); | 2603 | return simplelist_show_list(&info); |
2617 | } | 2604 | } |
2618 | #endif /* defined(HAVE_BOOTDATA) && !defined(SIMULATOR) */ | 2605 | #endif /* defined(HAVE_BOOTDATA) && !defined(SIMULATOR) */ |