FS#1656 - Enhanced menus + MDB patch (for recorder)

Attached to Project: Rockbox
Opened by Vincent Paeder (cent20) - Monday, 25 August 2003, 14:48 GMT
Last edited by Lee Pilgrim (diddystar5) - Sunday, 28 September 2003, 18:07 GMT
Task Type Patches
Category Settings
Status Closed
Assigned To No-one
Operating System
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


Used my widgets there...
This task depends upon

Closed by  Zakk Roberts (midkay)
Monday, 19 February 2007, 05:03 GMT
Reason for closing:  Out of Date
Additional comments about closing:  With all the recent menu changes, new targets, etc. this can be considered out of date/unusable. As for MDB, we\'ve had it in SVN for a couple years now.
Comment by Vincent Paeder (cent20) - Tuesday, 26 August 2003, 19:52 GMT

A good looking version. Text is now right aligned when a slider
is present (no more overlap between text and sliders then ...
as far as i know!)
Modified lcd-recorder.c to permit a per pixel scrolling line
positioning (fixes checkbox bug).
Added a new widget ("arrow"). Used in menus to indicate a
list of different values.
Comment by Lee Pilgrim (diddystar5) - Sunday, 28 September 2003, 18:07 GMT

I have been playing around with your patch today and I have
a few comments.

1. Could you please remove all the whitespace changes. They
make your patch outdated quicker, and harder to read.

P.S. Could you also fix that on the MDB patch?

2. Try not to include other patches in your patch. I noticed
you included the accurate vbr ff-rew and more directory
sorting options patches.

3. Why is the fade on pause and stop renamed? is dosen't
make any sense when it says "on Pause/Stop"

4. Your patch has some compile errors. Could you please fix
those. Also your patch dosen't compile with the simulator.

5. Could you please post up a patch without the MDB, and
just the menu changes.