Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category
  • 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 lowlight - 2006-03-07

FS#4796 - Display current dir while browsing files

This patch adds “titles” to the gui list widget. It allows for setting a title/icon to display at the top of the list.

Currently, I’m using this feature to show the current directory (except the root dir) while browsing files. See http://forums.rockbox.org/index.php?topic=2876.0 for an example. To enable this, there’s a menu option in General Settings → File View → Show Current Dir.

Note: Scrolling the title was a problem because line scrolling depends on the screen xmargin which needs to be different for the title line versus the rest of the items. I’ve included a work around for ONLY the H100 (lcd-h100.c and lcd-h100-remote.c) and I haven’t observed any fowl behavior. The changes should be the same for the other lcd files.

If this patch is accepted (or if there is interest), I can try to extend this concept to it to the menu system as well, i.e. to display the current menu.

Closed by  safetydan
2006-08-16 04:22
Reason for closing:  Out of Date
Additional comments about closing:  

I\'m sorry that we missed this patch when http://www.rockbox.org/tr acker/task/5795 got comitted. I\'ll close it now since it is out of date with the functionality in CVS.

bk commented on 2006-03-07 19:33

Works well! I think it’s a great idea to implement this for menus as well (they both use the list widget, so it shouldn’t be too hard to do).

Is it possible to highlight the title somehow? Display it in inverse or underline?

Shouldn’t the root dir have a title too? “/” ? If menus have titles this will be less of an issue.

I made a version of this patch which will apply and compile cleanly against paprica’s scroll margins patch (which can be found here : http://www.rockbox.org/tracker/task/2994 )
Use -p0 to apply

UPDATE: 2006-03-21
Synced to current cvs.

Your sync was broken by a commit which changed english.lang.
I resynched it with the current CVS (2006.03.22), and made a version which will apply cleanly against the scroll margins patch

BTW: I love your patch :) I hope it’ll get included in CVS soon…

Would it be possible not to display it on the remote because we already dont have a lot of lines displayed, so one less is quite annoying ;)

upadated for current CVS

fixed a small mistake in the previous patch and added a version which will apply and compile cleanly against the scroll margins patch.

I tried to patch using the list_titles_scrollmargins.patch posted on April 4, but it fails to patch properly with June 23’s CVS.

Can we get an update?

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing