Bug 1444 - Browsing an artist's albums in player UI unnecessarily includes artist name
: Browsing an artist's albums in player UI unnecessarily includes artist name
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: 6.1.0
: All All
: P2 minor (vote)
: ---
Assigned To: Vidur Apparao
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-04-26 13:18 UTC by Ben Sandee
Modified: 2008-09-15 14:37 UTC (History)
0 users

See Also:
Category: ---


Attachments
add the artist only when browsing album list at top level (507 bytes, patch)
2005-04-28 23:31 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Sandee 2005-04-26 13:18:20 UTC
On my SliMP3, likely due to to the new browse album behavior where the artist
name is included, I'm also seeing the artist included in the album name even if
we're already browsing by artist.  For example:

Browse Music -> Browse By Artist -> R.E.M. ->  Document (R.E.M.), Murmur
(R.E.M.), Automatic for the People (R.E.M.).

I can understand the desire to include the artist name when browsing by album,
in fact I applaud it, but it seems to be unnecessarily verbose on an already
cramped display (I use large font on my SliMP3's).

That said, given the tradeoff of either not having this feature at Browse By
Albums or living with the extra artist info in browse by artist, I'd definitely
go with including it both places because it's a great improvement.
Comment 1 KDF 2005-04-26 13:49:41 UTC
simple fix would be to include artist in level zero of the hierarchy only.
Comment 2 Ben Sandee 2005-04-26 13:57:17 UTC
I was thinking about that too.

Something to consider is that if someone were to add Browse by Year to the
player UI, then the albums/artists would not be at level zero (I think).
Comment 3 KDF 2005-04-26 23:39:23 UTC
yup...but that's not a concern until someone does :)  
This is always how it works with a bug.  it can be fixed now, or left until a
larger rework fixes it as a side effect.
Comment 4 KDF 2005-04-28 23:31:40 UTC
Created attachment 472 [details]
add the artist only when browsing album list at top level

simple patch as described in earlier comments.
Comment 5 Vidur Apparao 2005-04-29 07:46:36 UTC
Patch looks good. Check it in.
Comment 6 KDF 2005-04-29 17:31:13 UTC
checked into trunk at change 3106
Comment 7 Ben Sandee 2005-04-29 18:02:20 UTC
looks great kdf, marking fixed!
Comment 8 Chris Owens 2006-06-16 14:42:00 UTC
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006.  I am setting them to targets of 6.2.1 to keep them from showing up in my queries.