FS#10094 - Replace oldmenuapi menu/customed menu in plugins by standard menu

Attached to Project: Rockbox
Opened by Teruaki Kawashima (teru) - Saturday, 04 April 2009, 07:14 GMT
Last edited by Jonathan Gordon (jdgordon) - Friday, 03 July 2009, 04:44 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To Jonathan Gordon (jdgordon)
Operating System All players
Severity Low
Priority Normal
Reported Version Version 3.2
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


this patch replaces menu of the following plugins by standard menu.

plugins using oldmenuapi:

plugins using its own menu:

plugins whose menu is NOT replaced:
- this plugin has file browser and it needs some work to turn to standard menu.
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Friday, 03 July 2009, 04:44 GMT
Reason for closing:  Accepted
Additional comments about closing:  this has been open for long enough... if any bugs come up new tasks can be opened
Comment by Teruaki Kawashima (teru) - Wednesday, 20 May 2009, 14:27 GMT
updated patch.
removed unrelated changes and fixed wrong changes.
Comment by Teruaki Kawashima (teru) - Saturday, 23 May 2009, 13:10 GMT
attempt to replace rockpaint's menu by standard menu.
i'm not sure if i'm doing things correctly for browser, but it seems to work.
not done for font selection.
Comment by Teruaki Kawashima (teru) - Wednesday, 03 June 2009, 12:59 GMT
update patches and divide to some parts.

this patch replaces oldmenuapi by standard menu simply and won't have any issue.

this patch replaces oldmenuapi by standard menu with smoll change.
move "Quit" from top to button item of menu just like other plugins.
use simplelist_info for loading/saving game.
use differt menu wheter being in game.
make "Draw Three Cards"/"Draw One Card" to option "Draw Cards Option".

this patch replaces oldmenuapi by standard menu for doom.
this contains large change for creating item and selecting item for "Addons","Demos" witch may contain problem.

i'm not sure if changes relating to mpeg_menu_sysevent_callback is correct.

this is patch for replacing customed menu by standard menu.
Comment by Jonathan Gordon (jdgordon) - Wednesday, 03 June 2009, 16:25 GMT
are those all seperate patches that can be aplied in any order?
I cant see oldmenu.c being removed there.. does that mean not all plugins are converted after these? I'll look into commiting this before he weekend (hopefully)
Comment by Teruaki Kawashima (teru) - Thursday, 04 June 2009, 13:01 GMT
yes, they can. these patches are independent each other.
more strictly, independent against each plugin.
i converted all plugins using oldmenuapi unless i missed something.
the reason why removal of apps/plugins/lib/oldmenuapi.c is not included is that I thought that it's not late to remove oldmenuapi.c after these patches are commited and problems are not reported.
Comment by Teruaki Kawashima (teru) - Saturday, 06 June 2009, 03:51 GMT
I had uploaded wrong standard_menu_for_plugins_with_tweak.3.patch, sorry...
it dousen't do what i said for viewer.
> move "Quit" from top to button item of menu just like other plugins.
this is the correct one.
Comment by Jonathan Gordon (jdgordon) - Tuesday, 16 June 2009, 04:26 GMT
Accepted. Thanks....

I'm going to leave this open for a while in the hope that any problems that come up will get posted here.
Comment by Marko Pahlke (perfectdrug) - Tuesday, 16 June 2009, 11:07 GMT
I think it was more convenient to have the "quit" entry at the top of the list, in the jpegviewer, to easily exit the plugin without further keypresses, I think having "return" at the top is not intuitiv because if you press stop you normally don't want to undo this action.
Comment by Teruaki Kawashima (teru) - Wednesday, 17 June 2009, 14:47 GMT
"Quit" is at the bottom of the menu for (almost) all other plugins (if it has one), and it confused me that the "Quit" was not at where i expected (i.e. the bottom of the menu).
I'm wondering why there isn't key assigned to quit jpegviewer directly.
About "Return", maybe it can be dropped because user can return by pressing left?