Rockbox

Tasklist

FS#6082 - Switch order of playlist / sound settings in WPS Context

Attached to Project: Rockbox
Opened by Mike Miller (mikeage) - Thursday, 28 September 2006, 14:36 GMT
Last edited by Jonathan Gordon (jdgordon) - Saturday, 04 August 2007, 14:42 GMT
Task Type Patches
Category User Interface
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This patch will reverse the order of the playlist/playlsit catalog and sound settings menu items in the WPS Context Menu.

With this patch, to view the current playlist from the WPS, simply:

Hold select
Select
Select

Instead of

Hold select
Down
Select
Select.

See http://forums.rockbox.org/index.php?topic=6736.0

This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Saturday, 04 August 2007, 14:42 GMT
Reason for closing:  Rejected
Comment by Reza (afruff23) - Thursday, 12 October 2006, 04:42 GMT
This is much needed. The "view current playlist" menu item is too nested in the menus.
Comment by Mike Miller (mikeage) - Thursday, 12 October 2006, 04:45 GMT
It's still nested, but since it's only one unique button away, I think it's easier.
Comment by Ryan Sawhill (ryran) - Wednesday, 18 October 2006, 09:15 GMT
thank you thank you thank you. I really appreciate this patch. :)

(so obviously there's my vote for a commit.)
Comment by Steve Bavin (pondlife) - Monday, 30 October 2006, 16:57 GMT
I vote for a commit too.
Comment by Jon (ace214) - Monday, 05 February 2007, 02:27 GMT
does this still actually work?
Comment by Mike Miller (mikeage) - Monday, 05 February 2007, 02:33 GMT
Of course it does :)

It'd be hard to _break_ a patch this simple and stupid...
Comment by Jon (ace214) - Monday, 05 February 2007, 02:34 GMT
when i applied it just now, it patched but didnt function on the device
Comment by Mike Miller (mikeage) - Monday, 05 February 2007, 02:56 GMT
Hrmm.

I just checked svn diff against the latest version.. the patch looks the same, although the line numbers changed. Perhaps since there's relatively little context, it's not applying it in the right place.

Try the attached.
Comment by Jon (ace214) - Monday, 05 February 2007, 03:12 GMT
nope. didn't work.
Comment by Mike Miller (mikeage) - Monday, 05 February 2007, 03:19 GMT
Weird. Works for me on my ipod 4g gray... builds are available at mikeage.net/content/rockbox-custom-builds/

Do you see _any_ change at all?

When you press and hold select from the WPS, what menu items do you have?
Comment by Jon (ace214) - Monday, 05 February 2007, 03:30 GMT
OK, sorry. I was an idiot.... -_-
Comment by Mike Miller (mikeage) - Monday, 05 February 2007, 03:38 GMT
NP :)
Comment by Ryan Sawhill (ryran) - Monday, 05 February 2007, 03:46 GMT
nice. thanks for updating it Mike. :)
Comment by Ryan Sawhill (ryran) - Monday, 12 February 2007, 20:56 GMT
a simple synch
Comment by Ryan Sawhill (ryran) - Monday, 02 April 2007, 20:02 GMT
sync to svn13005
Comment by Ryan Sawhill (ryran) - Monday, 30 April 2007, 20:03 GMT
ridiculously easy to do now with the new menu system... but sure.. why not.

Loading...