FS#11250 - Set hotkeys from a normal menu, not with a key in the context menu

Attached to Project: Rockbox
Opened by Jeffrey Goode (Blue_Dude) - Saturday, 08 May 2010, 02:17 GMT
Last edited by Jeffrey Goode (Blue_Dude) - Sunday, 09 May 2010, 02:12 GMT
Task Type Patches
Category User Interface
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


This removes hotkey setting in the context menu and adds menu items for setting. The reset screen is gone, the view hotkeys screen is gone. The yesno dialog is gone. All that's left is two menus, one for wps and one for tree. I figured out a way to change the settings_list to only show the appropriate options, so that behaves.

The hotkey assignments are unchanged. The hotkey behavior is unchanged. Only the method of assigning them has changed.
This task depends upon

Closed by  Jeffrey Goode (Blue_Dude)
Sunday, 09 May 2010, 02:12 GMT
Reason for closing:  Accepted
Additional comments about closing:  r25905
Comment by Alexander Levin (fml2) - Saturday, 08 May 2010, 07:45 GMT
I like it! WHat about adding the item "Turn the backlight on"? This would be a "do nothing" function that would only make the backlight go on -- without the splash "hotkey not set".
Comment by Jeffrey Goode (Blue_Dude) - Saturday, 08 May 2010, 18:09 GMT
More cleanup, includes manual updates. Syncs with recent lang updates. Hotkey is now ignored if no assignment is set, instead of displaying a splash.

I haven't tried it yet, but I'll bet just turning the hotkey off will bring up the backlight upon a hotkey press without doing anything else.

Comment by Jeffrey Goode (Blue_Dude) - Saturday, 08 May 2010, 18:36 GMT
Use these lang files instead. I don't think I deprecated the other ones nicely.