Bug 12067 - Remove music note icons from tracklist (tracks in an album/playlist) and current playlist screens
: Remove music note icons from tracklist (tracks in an album/playlist) and curr...
Status: RESOLVED WONTFIX
Product: SqueezePlay
Classification: Unclassified
Component: Browser
: unspecified
: PC Other
: -- normal (vote)
: 8.0.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-05-15 11:41 UTC by Weldon Matt
Modified: 2011-01-14 10:07 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
new style for tracklist (42.51 KB, image/png)
2009-05-15 21:16 UTC, Ben Klaas
Details
remote skin tracklist (23.88 KB, image/png)
2009-05-15 21:20 UTC, Ben Klaas
Details
song search results page (56.93 KB, image/png)
2009-05-15 21:21 UTC, Ben Klaas
Details
controller skin (16.43 KB, image/png)
2009-05-15 21:21 UTC, Ben Klaas
Details
Ref artwork - proposal to integrate album artwork back into touch skin (66.47 KB, image/png)
2009-07-24 17:09 UTC, ndijulio
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Weldon Matt 2009-05-15 11:41:58 UTC
Noah, Dean and myself reviewed the latest change this week and decided it was a bit too much.  Let's remove the music note icons that appear to the right of each track name.

We'll keep the album cover art on the left, but remove the icons from the right (do not replace with carrots - just leave this area blank except for the checkmark for currently-playing track).

Sorry for the confusion, thanks for letting us test this.
Comment 1 Ben Klaas 2009-05-15 12:16:31 UTC
I'll take 'em out, but I actually disagree with this. If we have touch-to-play behavior, we should at least give the user a visual indication that's what is going to happen.

I thought blank was the way to go before I put the notes in, now I feel the exact opposite.
Comment 2 Weldon Matt 2009-05-15 12:29:39 UTC
I don't disagree.  I wanted play icons, not notes (the notes look way too busy/cluttered - they dominate the screen IMO, and I think Noah agrees).

For the record, Noah and I both wanted nothing.  (ipods and other media players work perfectly fine this way, with nary a complaint that I've ever heard).  But that wasn't an option.  The options we were given from Dean were either the notes or the album covers.  So we're going with the album covers.

Seriously though, this is one of those cases of anticipating a problem we don't know we have yet (users not knowing that clicking a song will play it).  I think this solution will be just fine.  All it takes is one click on a song to learn the behavior, and again, its basically the same treatment everyone else uses.
Comment 3 Ben Klaas 2009-05-15 12:36:12 UTC
okay. still disagree with removing them, but okay.
Comment 4 Blackketter Dean 2009-05-15 15:56:22 UTC
Note that the removal of the note symbol should accompany the inclusion of artwork for every song/track/station/podcast. 

In essence, we are removing the icon on the right and reinforcing with the icon on the left.

Let's see it and if we're wrong, it won't be that hard to revert.
Comment 5 Ben Klaas 2009-05-15 16:40:41 UTC
something to keep in mind is Controller. This change means tracklist pages will be reduced from 8 items per page to 4 items per page.

Regardless, I should be able to do this reasonably quickly.
Comment 6 Ben Klaas 2009-05-15 21:16:02 UTC
fixed SC r26645 and Fab4 r5762

controller ends up looking okay because the text_list window style doesn't deliver an icon for each item on that screen and the 'itemplay' style shows a filled arrow. I think (I hope) that's how we want controller to look.

will post screenshots here
Comment 7 Ben Klaas 2009-05-15 21:16:50 UTC
Created attachment 5225 [details]
new style for tracklist

when browsing into an album
Comment 8 Ben Klaas 2009-05-15 21:20:38 UTC
Created attachment 5226 [details]
remote skin tracklist
Comment 9 Ben Klaas 2009-05-15 21:21:15 UTC
Created attachment 5227 [details]
song search results page
Comment 10 Ben Klaas 2009-05-15 21:21:34 UTC
Created attachment 5228 [details]
controller skin
Comment 11 Blackketter Dean 2009-05-15 21:32:50 UTC
Thanks, Ben, this is great.  Thanks for making the changes for review.

I'm going to reopen this as a placeholder for fixing the track listings in the various skins.

It's a no-brainer to me that the search results look dead-on right.

The remote and controller skin track lists should have the artwork on the left (with no track number).

We'll also need to fix the layout of the title bar, but that's probably a different bug.
Comment 12 Jim McAtee 2009-05-16 10:08:09 UTC
(In reply to comment #7)
> Created an attachment (id=5225) [details]
> new style for tracklist
> 
> when browsing into an album

The others look fine, but thumbs down on artwork when browsing into an album.  The user just came from a screen showing the album cover, and in virtually all cases the artwork for every track is going to be identical.  It's redundant, takes up space, and looks odd.
Comment 13 ndijulio 2009-07-23 09:21:57 UTC
Matt- Where did we leave off with this?  The possibility for redundant artwork is a show stopper IMO.
Comment 14 Weldon Matt 2009-07-23 12:00:47 UTC
We're adding the notes back, but ONLY on the current playlist screen (separate bug for this I think).

Don't know if a final decision was made re: tracks within an album (in terms of album cover artwork).

Logic would dictate that those cover images aren't needed there, as Jim mentioned.
Comment 15 ndijulio 2009-07-24 17:09:40 UTC
Created attachment 5522 [details]
Ref artwork - proposal to integrate album artwork back into touch skin

This method would allow artwork to be seen within the touch album template without image redundancy.
Comment 16 Ben Klaas 2009-08-03 07:55:52 UTC
this could get very tricky because we aren't being consistent with our menu item behavior...in an album track list we don't show icons except for on this new first item, and in a search track list we do show icons on every item.

Noah, while I like your proposal visually, it's exactly what I've been trying to get away from, putting "special" items above and below lists of items served by SC.
Comment 17 Ben Klaas 2009-08-23 22:01:19 UTC
sorry, this is absolutely not a 7.4 release blocking issue. Pushing this all the way to 8.0
Comment 18 Ben Klaas 2011-01-14 10:07:25 UTC
what hasn't been done in this bug, won't be.