PDA

View Full Version : Patch for adding album art to listings



2005-01-21, 12:41
Matt Welsh wrote:

> I made a quick patch to add album art to the HTML artist and album
> listings in the SlimServer Web interface.

This is a great patch. I really like having the thumbnails in the album listings. An alternate approach (and one that I like a bit better) would be to simply use the "Browse by Artwork" form with its matrix of thumbnails in any album view. My sense is that it wouldn't take too many changes to Pages.pm to make this work. I had partial success by simply commenting out the lines that treat an artwork view and a regular album listing differently. A cleaner approach would be to have a server option to choose whether to "Display Artwork in Album Listings".

Taylor

kdf
2005-01-21, 12:51
Quoting trmp3 (AT) hoghill (DOT) net:

> Matt Welsh wrote:
>
> > I made a quick patch to add album art to the HTML artist and album
> > listings in the SlimServer Web interface.
>
> This is a great patch. I really like having the thumbnails in the album
> listings. An alternate approach (and one that I like a bit better) would be
> to simply use the "Browse by Artwork" form with its matrix of thumbnails in
> any album view. My sense is that it wouldn't take too many changes to
> Pages.pm to make this work. I had partial success by simply commenting out
> the lines that treat an artwork view and a regular album listing differently.
> A cleaner approach would be to have a server option to choose whether to
> "Display Artwork in Album Listings".

I agree, it is a fantastic patch. It was always a desire to have the option of
displaying many of the browse lists with artwork of some kind. What would be
great would be if this patch can be made to work with the 6.0 set of code.
5.4.1 is effectively closed to any new features, even tiny ones. A great deal
of the older Info API has been moved to the DataStores API so its not a trivial
thing to move somethign like this to 6.0, but it would be very nice to see it
done.

cheers, and great work!
-kdf