Rockbox Development > Feature Ideas

PictureFlow Improvement

<< < (3/5) > >>

pabouk:
Of course directory cache slows down startup significantly. That is the reason why I wrote it as just an unimportant additional secondary argument for users which already use directory cache. Maybe it was not clear.

I do not know any statistics but for me personally the directory cache adds so much to the player usability that I do not hesitate to enable it. Indeed the longer startup is unpleasant but it is just a little price for the advantages during runtime.

csavery:
I just saw this topic and it's related to what I wanted.

So I went ahead and coded this now for my own use and have it working here.

I haven't made a patch yet but if others want an WPS integrated PictureFlow I'd be happy to make a patch.

What I've done:

Added a setting to PictureFlow for "Auto WPS". This causes PictureFlow to goto to WPS when an album is played (with Set key). It also saves a "last_album" value so it knows where to come back to.

Added a PictureFlow WPS Hotkey setting. This causes the * hotkey in WPS to load PictureFlow directly (which now remembers which album it was centered on). I wanted to make it work with generic plugins but the Hotkey menu doesn't currently support a filename parameter, so I hard coded PictureFlow.

It works quite well. You can pretty much instantly bounce back and forth between PictureFlow and WPS to use it as album selector. I'm using it but haven't given it a thorough testing. It's fresh still.

As for autorun, well, if you boot into WPS then PictureFlow is just one key to select a new Album. Ta Da.

Oh, I had to add a new language string for the hotkey setting as well.

Vague Rant:
So is that to say that the track list display from PictureFlow is unavailable as a result of our changes? It sounds interesting, not particularly useful for me as a Clip+ user (monochrome screen makes PicFlow a bit ugly) but definitely something I'd be willing to test out.

EDIT: Meantime, there are a few improvements which could make PictureFlow a lot more usable on small/mono screens like the Clip series.



Here's how I have it set up at present, but there are several issues:

1. The "show album title" setting is not saved, and simply defaults to displaying at bottom at all times. The Clip series work best with the title displayed at top, as the top 16px are yellow/black (and the bottom 48px are blue/black).

2. The covers are practically unusably small; I have them zoomed out to 51% and that's the reason for their size; the reason for this is that otherwise the covers overlap with the yellow section, which creates a jarring disconnect both in colour and spatially--there's a ~2px "gap" between the yellow and blue areas. Two things might assist with this; firstly the images need to be "forced" to remain in the bottom 48px, secondly it'd be nice for all targets if there were an option to disable the reflections and allow the covers to take up that space; certainly the reflections are pretty on devices which can display them meaningfully, but the increased size may well be preferred on all devices.

csavery:
Vague Rant: keep in mind that my changes are just something I've done for myself and are not merged with the general Rockbox version. I do have a patch now which can be done against the latest svn build.

Regarding track list I chose to have "select" first click go to track list and second click start playback and go to WPS. I was thinking about having it go direct on first click and decided to play with it for a while and see what I preferred. I've also added another small change so that when a playlist ends or is stopped the WPS reverts back to the PictureFlow rather than dumping you into the tracklist as usually happens. This is because otherwise to get back to PictureFlow you have to go via the main menu again since you can't get to WPS when nothing is playing.

saratoga:
If you haven't already, you probably want to go talk to kugel on IRC about that patch.  Hes the most recent person working on pictureflow. 

Navigation

[0] Message Index

[#] Next page

[*] Previous page