Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#1973 - Onplay added to wps

Attached to Project: Rockbox
Opened by Henrik Backe (hbacke) - Thursday, 29 January 2004, 17:35 GMT+1
Last edited by Henrik Backe (hbacke) - Wednesday, 17 March 2004, 21:28 GMT+1
Task Type Patches
Category
Status Closed
Assigned To No-one
Player Type
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Onplay is added to wps.
This also enabled several changes in the main_menu.

- Create directory moved to onplay,
- Browse id3 moved to onplay
- Playlist moved to onplay
- Bookmarks moved to onplay.
- Sleep timer move to the root of the main menu
- Mange settings moved to the root of the main menu.

playlist_menu.[ch] can be removed. Everything
in playlist_menu.c is moved to onplay.c.
This patch only "if 0" everything in playlist_menu.c

.../Henrik


   onplay.diff (22.5 KiB)
 bookmark.c        |   35 -------
 bookmark.h        |    2 
 main_menu.c       |   99 ++++++++++++++++++---
 onplay.c          |  251 +++++++++++++++++++++++++++++++++++++++++-------------
 onplay.h          |    2 
 playlist_menu.c   |    2 
 playlist_viewer.c |    3 
 screens.c         |   13 ++
 settings_menu.c   |   82 -----------------
 tree.c            |    4 
 10 files changed, 301 insertions(+), 192 deletions(-)

This task depends upon

Closed by  Zakk Roberts (midkay)
Monday, 19 February 2007, 06:10 GMT+1
Reason for closing:  Out of Date
Additional comments about closing:  Seems like most or all of this is already in SVN (context menu for all devices). Anything that isn\'t in SVN now is out of date and probably Recorder-only.
Comment by Björn Stenberg (zagor) - Friday, 30 January 2004, 11:12 GMT+1

Looks good. I'd say we can start using F3 instead of ON+PLAY
on Recorders, in accordance with the NEWKEYS document. (Just
remember players don't have F3, so they will keep ON+PLAY).

I'm thinking about maybe releasing a 2.2 before we commit
the full UI overhaul though.
Comment by Henrik Backe (hbacke) - Friday, 30 January 2004, 11:35 GMT+1

Ok, I'll do that for both wps and tree browser,
should I also the on+play combo for the recorder ?

.../Henrik
Comment by Björn Stenberg (zagor) - Friday, 30 January 2004, 12:36 GMT+1

If it's just a few lines of code, keep it for recorder too.
Comment by Henrik Backe (hbacke) - Friday, 30 January 2004, 12:49 GMT+1

Patch updated with f3 keys. on+play combo kept for the
recorder (very few lines of code).

.../Henrik
Comment by Henrik Backe (hbacke) - Friday, 30 January 2004, 19:16 GMT+1

Correct version uploaded.

.../Henrik
Comment by Henrik Backe (hbacke) - Saturday, 31 January 2004, 21:44 GMT+1

And a a working version .-)

.../Henrik
Comment by Henrik Backe (hbacke) - Monday, 09 February 2004, 16:09 GMT+1

Fixed a set_margin bug.

.../Henrik
Comment by Henrik Backe (hbacke) - Saturday, 13 March 2004, 23:16 GMT+1

Updated to latest cvs.
Uses c0utta's context defines so it is a context
sensitive menu. It is not bound to f2 any more
because I have not had time to figure out how to
easily inplement it in the new quickscreen screen.

.../Henrik
Comment by Henrik Backe (hbacke) - Monday, 15 March 2004, 21:10 GMT+1

Updated to latestcvs.

.../Henrik
Comment by Henrik Backe (hbacke) - Tuesday, 16 March 2004, 22:11 GMT+1

Updated to latest cvs.

.../Henrik
Comment by Henrik Backe (hbacke) - Wednesday, 17 March 2004, 21:28 GMT+1

Updated to daily cvs

.../Henrik

Loading...