FS#6087 - Some debug menu options do not allow left to exit

Attached to Project: Rockbox
Opened by Steve Bavin (pondlife) - Friday, 29 September 2006, 09:28 GMT
Last edited by Steve Bavin (pondlife) - Thursday, 02 August 2007, 10:49 GMT
Task Type Bugs
Category User Interface
Status Closed
Assigned To No-one
Operating System All players
Severity Very Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


For consistency, it would be good if all options from the menus allowed LEFT to return to the previous menu - most do, the following do not:
- View battery
- View partitions
- View disk info
- View runtime

Also it would be better if Screendump had an Enabled/Disabled submenu, like other settings.
This task depends upon

Closed by  Steve Bavin (pondlife)
Thursday, 02 August 2007, 10:49 GMT
Reason for closing:  Rejected
Additional comments about closing:  I was obviously in a picky mood when I opened this.
Comment by Dominik Riebeling (bluebrother) - Friday, 29 September 2006, 09:58 GMT
Recently I noticed that e.g. the "View I/O ports" can now be exited with Left. I think the debug menu is special, so it would be better to leave all debug menu screens with Stop only. Right now, the ADC value for the left button won't get shown anymore as Left leaves the screen.
Comment by Rani Hod (RaeNye) - Friday, 29 September 2006, 12:49 GMT
What I'd like to see is some hierarchy in the debug menu.
Some items are HW specific, some are RB-wide.
It'd be easiest to have something like: (taken from X5 debug menu, might vary for others)

-dump ROM
-catch mem access

-os stack
-audio thread

-- CPU freq
--HDD info
-FM radio
-USB-OTG (just thinking ahead...)
Comment by Jonathan Gordon (jdgordon) - Wednesday, 27 December 2006, 07:22 GMT
ok, well I've just commited a fix for bluebrothers problem, as for the actual bug report, the problem is the action contexts being used have a different action for the LEFT button, so STD_CANCEL isnt being triggered, and there is no nice (and target independant) way to fix this.

As for re-ordering the menu options. I think everyone will agree that this menu really isnt important and so it doesnt matter if its not neat.