Bug 17442 - Use Text only/Small Artwork/Large Artwork setting only for album lists
: Use Text only/Small Artwork/Large Artwork setting only for album lists
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.6.0
: PC Windows XP
: P3 normal with 2 votes (vote)
: 7.6.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-08-13 15:29 UTC by Jim McAtee
Modified: 2011-08-26 01:27 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jim McAtee 2011-08-13 15:29:12 UTC
Somewhere during the onebrowser work, this setting began to be applied to track lists. It doesn't work well. A user wanting to control how album lists are displayed may not want tracks displayed the same. And you pretty much never want the large artwork grid (gallery) for a track listing. 

In the browse window, there are two options for the display of track lists:

1. Text only all of the time. This is how it was prior to 7.6.0.

2. Exactly like it's handled in the current playlist. The options should be 'show' and 'hide' artwork, and use the same sized small cover thumbnails and title formatting with between one and three lines of text. To do this, there needs to be another setting/cookie that controls this formatting only within the browse window. The drop-down list would then be displayed at the foot of any browse page showing a track listing.
Comment 1 Ian Pallfreeman 2011-08-14 04:16:29 UTC
I think this one's already been fixed (r33073) in various places, but similar "stacked" artwork is still appearing here and there, e.g. search display, last.fm plugin.
Comment 2 Jim McAtee 2011-08-14 04:26:39 UTC
My contention is that if an option to display small artwork for tracks is desired then there needs to be a separate setting for it, otherwise only display text. The text/small/large art option under browse albums was never intended to control the behavior of how track lists are displayed.

Fixing the 'stacked' artwork would only be fixing something that was implemented before being thought through.
Comment 3 Michael Herger 2011-08-14 22:12:10 UTC
*** Bug 17446 has been marked as a duplicate of this bug. ***
Comment 4 Michael Herger 2011-08-14 22:13:41 UTC
Unfortunately there's no generic "this is a track list only" flag which would work across all uses (eg. remote services). I'll see what I can do.

for my own reference: bug 17446 - track results in search
Comment 5 Jim McAtee 2011-08-14 22:20:15 UTC
If this can't be addressed now, then bug 17446, with overlapping/mis-sized artwork needs to be fixed in the meantime.
Comment 6 SVN Bot 2011-08-15 02:30:32 UTC
 == Auto-comment from SVN commit #33085 to the slim repo by mherger ==
 == http://svn.slimdevices.com/slim?view=revision&revision=33085 ==

Bug: 17442
Description: improve handling of track-style items in XMLBrowser code. Hide their artwork if possible.

Jim et al.: please let me know if this fixes all/most occurences of artwork showing up in places which don't make sense. I verified this change with the cases mentioned (search, last.fm), but can't do a full run-through of all apps & browse modes. Thanks!
Comment 7 Michael Herger 2011-08-25 06:55:13 UTC
Can this be considered fixed in the lateste nightlies?
Comment 8 Ian Pallfreeman 2011-08-25 07:17:58 UTC
Yes. But I still think removing my picture from last.fm is a cheat. :)
Comment 9 Michael Herger 2011-08-25 07:54:56 UTC
Thanks!
Comment 10 Jim McAtee 2011-08-25 11:13:12 UTC
I still see the following two things. Somewhat related, but maybe not part of this bug.

- When 'Text only' is in effect, the 'All Songs' link has an icon.

- When 'Small Artwork' is in effect, basic search results shows small covers for albums. This seems a little odd because you have no art for the artists or tracks,  and no covers when 'Large Artwork' is in effect, but I suppose it could also be considered the desired behavior.
Comment 11 SVN Bot 2011-08-26 01:03:48 UTC
 == Auto-comment from SVN commit #33210 to the slim repo by mherger ==
 == http://svn.slimdevices.com/slim?view=revision&revision=33210 ==

Bug: 17442
Description: suppress all artwork when we're in text only mode (irritatingly this is "artwork = 2"...)
Comment 12 SVN Bot 2011-08-26 01:27:56 UTC
 == Auto-comment from SVN commit #33211 to the slim repo by mherger ==
 == http://svn.slimdevices.com/slim?view=revision&revision=33211 ==

Bug: 17442
Description: disable artwork in search results