Rockbox

Tasklist

FS#10093 - Center on loaded file when browsing

Attached to Project: Rockbox
Opened by Jonas Häggqvist (rasher) - Friday, 03 April 2009, 15:42 GMT
Last edited by Jonas Häggqvist (rasher) - Monday, 22 June 2009, 18:02 GMT
Task Type Patches
Category User Interface
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Version 3.2
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This patch centers the browser on the currently loaded font when doing "browse fonts".

Something smarter should probably be done that does the same for themes, languages and wps files.
This task depends upon

Closed by  Jonas Häggqvist (rasher)
Monday, 22 June 2009, 18:02 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed in r21464
Comment by Jonas Häggqvist (rasher) - Tuesday, 16 June 2009, 09:56 GMT
Updated patch that also works for languages, except when no language is selected, in which case it should use English, but doesn't currently.
Comment by Jonas Häggqvist (rasher) - Monday, 22 June 2009, 15:47 GMT
Fixes:
- Now picks English when no language has been set.
- Accidentally set the "current" variable static, which meant it was set even when none of the conditions were true (eg. when browsing themes)
Additions:
- Works for FM presets
- Works for WPS
- Works for RWPS
Remaining bugs:
- set_current_file() does something to the dirbrowser which means you don't exit it when you press back, so when you browse fonts, and press back, you're in .rockbox, and back again brings you to the root. Only on the third press on back are you taken to the menu.
Comment by Jonas Häggqvist (rasher) - Monday, 22 June 2009, 15:52 GMT
Fixes
- Reset dirlevel to 0, so pressing back returns to the menu.
Remaining bugs:
- None! :-)
Comment by Jonas Häggqvist (rasher) - Monday, 22 June 2009, 16:17 GMT
Commit candidate:

Fixes:
- Remove the DEBUGF
Additions
- Change "Browse Fonts" into "Font"
- Change "Browse .wps files" into "While Playing Screen"
- Change "Browse .rwps files" into "Remote While Playing Screen"
- Change "Browse Themes" into "Browse Theme Files"
Comment by Jonas Häggqvist (rasher) - Monday, 22 June 2009, 17:29 GMT
Second commit candidate, I'll probably commit it shortly.

Additions
- Update the manual according to the menu wording changes.

Loading...