• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category User Interface
  • Assigned To
  • 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 learman - 2007-05-12
Last edited by learman - 2007-05-26

FS#7158 - Bookmark selection as a list.

This is a first, rough implementation of having the bookmark selection screen as a list. It's not finished, but it is complete enough to be usable. A proper version would include at least the following:

* Properly hooked up delete bookmark button.
* Optimized bookmark file reading (currently the file is opened, scanned and closed twice for each displayed item).
* Proper lang strings.

And perhaps the following:

* Merge the "Load bookmark?" screen with the list (first item would be "Don't resume", with second item selected on entry).
* Context menu for Select bookmark, Delete bookmark, Exit screen.
* Different display of the bookmark items.
* Different voicing.

Originally I had though of indicating shuffle mode using the list icon, but that icon is currently not available to the list, so I added the "Shuffle" text instead.

Comments and suggestions are appriciated.

There's also a screenshot, showing what it can look like.

Closed by  learman
2007-05-26 10:45
Reason for closing:  Accepted

Second alpha version. Changes:

* Small changes to bookmark display.
* Don't scroll all lines.
* Simple bookmark file buffering implemented, using the plugin buffer.

The buffering is quite simple, so list wrapping will always cause a reload (unless the file fits in the buffer). However, the plugin buffer should be larger that most bookmark files, so this shouldn't be a big problem.

First beta version. Changes:

* Context menu added (for Resume and Delete).
* "Load bookmark?" screen merged with bookmark list.
* Button mappings updated.
* English.lang updated.
* Buffering tweaks.

This is pretty much what I had in mind, but suggestions are still welcome.

I use my DAPS a lot to listen to audio books and as such bookmarking is one of Rockbox's killer features for me.
So I installed your patch and have been testing it this afternoon, so far I am impressed and find it to be quite a
big improvement over the old bookmark screen.

The only thing is that I find the the playlist position number a bit confusing as I feel it should really start at
one not zero.

I also think it would be clearer what the number means if it was displayed as [Playlist track number / number of tracks in play list]
like [64/100].

Currently it just displays the number in the bookmark file, which apparently is zero-based. :) Easily fixed.

Getting number of tracks requires reading the playlist/directory. As it only needs to be done once, that's not a problem. However, it would be repeated for each bookmark, which I don't quite like… Making it clearer that it is the index would be nice though.

I was under the impression that you would only have to re-read the directory structure more than
once for the resent bookmarks screen, as normal bookmarks files would have the same number of entry
in the directory for play list for all entries.

But I quite agree if its going to slow things down a lot them its most likely not a good idea.

I hadn't thought of the "Recent bookmarks" screen (I don't really use that one myself); there the entry count would indeed be different (and perhaps useful). But as you say, scanning every playlist/directory could make things slow.

New version:

* Synced with svn.
* Fixed display of playlist position.
* Some more buffering tweaks and fixes.

Should be ready for commit soon…

Looks good.

I have tested the lasted version of your patch on both my h140 and my ipod nano, so when do you plan to commit it :)


Available keyboard shortcuts


Task Details

Task Editing