Bug 11482 - Indication of currently playing track on playlist not obvious
: Indication of currently playing track on playlist not obvious
Status: CLOSED FIXED
Product: SB Touch
Classification: Unclassified
Component: Now Playing
: unspecified
: PC Windows XP
: P1 normal (vote)
: 7.5.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-03-25 16:32 UTC by Mickey Gee
Modified: 2010-04-08 17:25 UTC (History)
6 users (show)

See Also:
Category: ---


Attachments
current playlist window as of r6912 (73.12 KB, image/png)
2009-08-04 21:39 UTC, Ben Klaas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mickey Gee 2009-03-25 16:32:01 UTC
I selected My Music->Artist->The Alan Parsons Project->Play all to create a playlist consisting of 4 CDs. When I look at the playlist using the button on the upper right corner of the Now Playing screen (E.g., 14 of 60), the currently playing track was denoted by larger and bold font for track name, album name, and artist name. No album art for this track. It's not obvious that it denotes the currently playing track.

I'm sure it's going to change so this is just to create a reminder.


SC Version: 7.4 - 25564 @ Tue Mar 17 01:06:28 PDT 2009, Fab4 PB2 r4875 firmware
Comment 1 Jim McAtee 2009-06-08 14:00:37 UTC
I believe this has been fixed.  A music note is now displayed at the right of the current item in the playlist.
Comment 2 ndijulio 2009-07-15 15:13:09 UTC
Please reopen if necessary.
Comment 3 Jim McAtee 2009-07-15 15:26:28 UTC
I don't see a note on the current item.  Did this unfix itself or was I mistaken?
Comment 4 Weldon Matt 2009-07-15 15:34:39 UTC
I didn't repoen it, but I'm running r6488, playing a "my music" track and looking at the Current Playlist.

There is currently no "note" icon to the right of the currently playing track.  We need to add this back in...
Comment 5 Jim McAtee 2009-07-15 15:43:39 UTC
One other minor bug with the currently playing item is that the thin separator line at the bottom of the item disappears.
Comment 6 Ben Klaas 2009-07-16 08:57:21 UTC
fixed, r6615

Noah, can you review the assets for icon_nplay* for Fab4Skin? I have three, two of which are called icon_nplay_3line... are those the intended assets for selected/not selected?

If the assets are correct, go ahead and close this.
Comment 7 ndijulio 2009-07-16 19:18:10 UTC
Hi Ben,

Please verify that we are using the Now Playing icon across all platforms.  I have verified this in Fab4 only.  Couple of issues:

1. Assets for 5 line (touch) skin should point to "icon_nplay_off.png"

2. The padding needs to be increased away from the right edge.  Will provide guidelines ASAP.

3. The divider line (tch_5line_divider.png) is not displayed on the item below the current playing track.
Comment 8 Blackketter Dean 2009-07-22 09:02:07 UTC
Moving to the product SqueezePlay because this bug appears to apply to any player based on that application code.  Feel free to move it back if it's specific to the original product.
Comment 9 Ben Klaas 2009-08-04 21:34:40 UTC
 == Automatic comment from SVN commit #6912 to Repository: jive ==
https://svn.slimdevices.com/jive?view=revision&revision=6912

Bug: 11482
Description: fix layout in QVGA skins


 == Modified 1 files total ==
7.4/private-branches/fab4/squeezeplay/src/squeezeplay/share/applets/QVGAbaseSkin/QVGAbaseSkinApplet.lua
Comment 10 Ben Klaas 2009-08-04 21:38:38 UTC
I think we're in fine shape on this bug, but I'm not sure I've addressed all of the items in comment#7. Noah, I don't think I have an up-to-date guideline for the playlist window. Remember that these are "touch-to-play" items, so they don't get the > symbol on the right.

It's a separate discussion, but if there's anyplace where touch-to-play is frustratingly too easy to interrupt your current listening, it's on this window. If it were up to me current playlist items would pop a context menu by default on touch.
Comment 11 Ben Klaas 2009-08-04 21:39:11 UTC
Created attachment 5567 [details]
current playlist window as of r6912
Comment 12 Jim McAtee 2009-08-04 21:47:23 UTC
It looks like the divider line is still missing.
Comment 13 Ben Klaas 2009-08-14 08:09:17 UTC
don't understand where that line went, there's nothing in the skin that removes it in that style.

retargetting as 8.0 as this bug is now Fab4-specific and does not affect other UIs.
Comment 14 SVN Bot 2009-10-15 12:40:46 UTC
 == Auto-comment from SVN commit #7867 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7867 ==

Fixed Bug: 11482 +1
Description: redefine bgImg for albumcurrent style to show pencil line effect 
(not sure why this is necessary to redefine as it should inherit from s.item_play, but this fixes it)
Comment 15 Chris Owens 2010-04-08 17:25:05 UTC
This bug has been marked fixed in a released version of Squeezebox Server or the accompanying firmware or mysqueezebox.com release.

If you are still seeing this issue, please let us know!