Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by amar - 2006-05-05
Last edited by darkkone - 2006-06-07

FS#5294 - Quick exit for menu screen

press play in the menu and it will quit to the previous screen (I have been wanting this for a while, not sure if it counts as a bug though)

Though ATM it is only set up for H300 and H100, it just needs button asignments for the other targets

Closed by  pondlife
2007-03-02 10:22
Reason for closing:  Out of Date
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

I believe the root menu UI changes have superceded this patch. Re-open if I\'m wrong!

thinking of the button assignment in the menu on my h120 I think it’s a bit strange Play doesn’t do anything and Stop goes one level back. Similar to the file browser I think it would be good (and somewhat consistent) to have it the same in them menu:
- Play to return to wps
- Stop to stop playback
- Left to go a level up (as it already does)

amar commented on 2006-05-05 19:30

- Play to return to wps (this is what the patch does)
- Stop to stop playback (personaly I would like the oposet, stop to take you up one level but it doesn’t bother me)
- Left to go a level up (as it already does)

amar aka Pi

amar commented on 2006-05-16 14:08

Hi updated it a fair bit, now it should exit from all parts of the menu (including browse plugins etc)
Please test/post comments thanks
(as before it is atill only maped for the H300 and H100 main unit but adding remot and other targets should be simple

Following the discussion in IRC, my suggestion would be to do the following:

1) Implement a “quick exit” button in the menus - this would (where possible) be the same button used to enter the menu. So for example, on the h1x0 it would be BUTTON_MODE, and on the ipods, BUTTON_MENU.

2) Make the PLAY button resume playback (or take the user to the WPS) in more places in Rockbox - an extension of this patch could be used to implement that in the menus.

i.e. don’t try and mix the two features.

amar commented on 2006-05-16 16:31

Thanks

OK I have changed the assignments to the same button used to enter the menu and added support to all platforms however the ipods use BUTTON_MENU in the colour picker and in select.h and I don’t know what should be used instead. I _think_ remote support should be working now.

I only have an H300 <no remote> so if others could test the other platforms I would be grateful

amar commented on 2006-05-17 16:15

Now implemented part 2 of your suggestion, now when you quit the menu it always returns you to the WPS if music is playing and the file tree if no music is playing.

Press play, if no music is playing it resumes play list, then does a quick return.
Press stop, it stops current playback
Press menu, does a quick return

Buttons mapped on other targets based on the mappings in the file tree
Would appreciate it of people could check on other targets particularly the ONDIO (I have a bad feeling about the stop play in the file tree with that one)

amar commented on 2006-05-19 15:45

One more go, I think it covers everything asked for
Still not tested on any other targets!

Tested on iPod Video 60 GB. The patch applies and builds cleanly, and all functions appear to work as designed.

Hi all!
I have added this patch to my experimental builds for h300, but I don’t get any special action. Now I can press the A-B button (Menu button I assume) to get one step up, but pressing “Play” does not jump to the WPS screen. Any ideas about this?
Best wishes
Norbert
(for my builds, see http://www.misticriver.net/showthread.php?t=42675

Here is an updated patch for current cvs (made on 2006-06-12 20:00 +0200).

Similar as suggested on the mailing list, it would also be nice to have
- press rec (short) to enter the playlist. Rockbox is playlist based, and atm switching / showing the playlist is a bit clumsy – rec doesn’t do anything at all atm.
- press rec (long) to enter the recording screen. Not to any importance to me as I seldomly use recording, but it would be nice, and I guess really useful for the users that use recording frequently.
I think this would make the button behavior even more consistent.

(the above does of course only apply on targets with a dedicated rec button, in my case h120)

@bluebrother: I suppose this should be a different task (or maybe two) as it is an addition to wps and not to the menu handling.
Anyway here is my patch for “press rec (short) enters current playlist:

Ricewind: I think this is somewhat related to this task as I thought also on handling it from within the menu and file browser – mainly having toggle buttons for the playlist, so mode toggles the menu, rec the playlist, play the wps. Moving and selecting is completely handled by the “joystick” (haven’t looked on this for other targets than h100 yet).
But you’re right, it doesn’t only affect the menu screen so it isn’t related only to this task. Maybe we need a new task “make button handling more consistent” if we want to cover all this ideas in a single task (or two different tasks ;-)

I only just noticed this patch and I love it. The only problem I have found with menu_buttons_NP20060612.patch so far is that if you enter a menu from the radio screen and press the play button there, it will bring you to the WPS as expected with audio resuming and radio stopping - however the peakmeter is not working and I haven’t found a way to make it start work again without a reboot. Cheers, Roolku

amar commented on 2006-07-18 16:20

OK as the freeze is over i have updated against CVS again and added the fade(o); a nd bookmark fuction calls that were missing. Please check on other other targets (particularly ONDIO still not shure if i got it correct here

I’ve noticed one small bug. When in the menu system, the play button returns to the WPS, but it also pauses playback, which I believe is unintended. I have tested this on an iPod 5g using the most recent patch above. I have not yet had a chance to test on any other target.

amar commented on 2006-07-18 19:20

This doesn’t happen on my h300, so without an ipod to playwith, i can’t think of any way to fix this.
Anyone?

ryran commented on 2006-07-19 01:58

Oh wow. I always wondered about that, but just assumed it was intended.. for some reason.

I’ve tested this patch on Archos platforms on the sim, and it compiles and appears to work fine on all platforms except for the Archos Player, which breaks the compiler with the following error:

menu.c:209: error: `BUTTON_OFF’ undeclared (first use in this function)
menu.c:209: error: (Each undeclared identifier is reported only once
menu.c:209: error: for each function it appears in.)

This patch corrects the problem noted in my previous comment.

This patch now compiles on all platforms.

Project Manager

I liked this patch for a while, until you started adding more stuff than just the quick exit.

just a note: this patch makes it very hard to enter the id3-info screen from the wps (on irver with PLAY + A-B). you have to release both buttons at the exact same time, otherwise the screen is only shown for the time the buttons are pressed and is exited directly afterwards.

on my h120 I usually access the id3 screen via the wps context menu. I wasn’t aware there is this button combination. Is this intended? Also, I have this “very hard” behaviour also without this patch – when holding A-B too long the quick screen gets triggered sometimes, when holding play a bit too long it returns to wps. Also, sometimes the status bar (enabled, disabled in wps) doesn’t show up correctly.
Accessing the id3 screen from the context menu (as I’m used to) works without any problems.

yes, this button combo is intended. I only discovered it by examining the gwps.c source, I didn’t know it for a long time. But it is very handy to take a short look at your replaygain values.
If my problems aren’t related to your patch, then I suppose I just had more shaky hands today than usual (I was using a build without your patch for some time up to yesterday).

I updated the patch for CVS HEAD with all the new button changes. I modified the iriver h1x0-h3x0 keymaps so that
1. Main unit ON will start playback in the menu.
2. Main unit OFF will stop playback in the menu.

The updated keymap generates ACTION_WPS_STOP and ACTION_WPS_PLAY for OFF and ON. I modified menu.c to recognize those actions. I’m not sure whether this is appropriate or whether ACTIONs should have been added to action.h (e.g., ACTION_MENU_PLAY, ACTION_MENU_STOP).

I did not adjust any other keymaps.

NOTE: In CVS HEAD, there is no way to move up a level in the list tree using the remote. I submitted patch 5830 to correct that problem. I did not include that fix in this patch.

Updated for CVS HEAD. Verified it builds for all targets and sim targets.

ryran commented on 2007-01-07 07:25

any chance someone more knowledgeable than me could fix this bad boy? it’s pretty out of synch…

amar commented on 2007-01-07 10:40

I gave up trying to keep this synced when it became apparent that it was not going to be included in CVS. Currently my computer can’t compile rockbox but this may give me a reason to try again if i have some spare time.

I would also like to see an update of this useful patch!
The last time I used it I removed some code from the patch that wasn’t needed with the new button action system but I don’t remember if this was before the last update here or not. Unfortunately I don’t have those sources anymore. If you don’t get around to it in the next time I might update this patch (only if it is not very difficult and I need to have the time, too)

Here is a sync against todays cvs. This is more or less the patch I am using in my build. Only compile time tested.

Hunk error with today’s SVN. Hunk#1 failed at 509.

sorry- typo. didn’t compile. this one works.

The patch is seriously broken against SVN right now..

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing