Rockbox

Tasklist

FS#5994 - Mark current font/wps when viewing all (resolution for FS#5080)

Attached to Project: Rockbox
Opened by Alexander Levin (fml2) - Thursday, 14 September 2006, 20:51 GMT
Last edited by Jonathan Gordon (jdgordon) - Wednesday, 01 August 2007, 14:16 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

Position cursor to the current setting when browsing fonts, wps, rwps, eq preset, fmr. This can't be done for themes since the current theme is not saved in the settings.
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Wednesday, 01 August 2007, 14:16 GMT
Reason for closing:  Rejected
Comment by Alexander Levin (fml2) - Friday, 15 September 2006, 20:20 GMT
Updated the patch:
- introduced constants for standard file extensions
- replaced strcpy+strcat with snprintf (it's safer) -- thanks Linus for the tip
- moved some constants into format string (they are not printed with %s anymore)

The one who will (hopefully) commit this patch: please adjust the definition of FONT_DIR, LANG_DIR, and CODECS_DIR in settings.h so that it contains ROCKBOX_DIR (like e.g. WPS_DIR). Places where these constants are used should be also adjusted.

Linus, my patch doesn't break the 'Follow playlist' feature -- testes with the sim and on target (iriver H1xx).
Comment by Rani Hod (RaeNye) - Monday, 25 September 2006, 16:29 GMT
I was thinking that maybe after jdgordon commits the new settings menu patch, it'd be better to have this done in a similar way (i.e., have a little arrow pointing to the current setting as well as beginning on it).

Loading...