Bug 14665 - Implement final design of Fab4 NP screen
: Implement final design of Fab4 NP screen
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: Now Playing
: unspecified
: PC Other
: P1 normal (vote)
: 7.5.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-10-08 09:29 UTC by Ben Klaas
Modified: 2010-04-08 17:26 UTC (History)
4 users (show)

See Also:
Category: Feature


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Klaas 2009-10-08 09:29:25 UTC
placeholder for work needed on v2 now playing for 7.5.0 release
Comment 1 Ben Klaas 2009-10-16 09:11:32 UTC
*** Bug 13383 has been marked as a duplicate of this bug. ***
Comment 2 Ben Klaas 2009-10-30 11:34:27 UTC
This isn't really a NPv2 bug now, but a "here's what we're going to do on touch for 7.5.0".

notes:

3'UI:
1. playlist button should switch to + button when playlist size is 1
2. (possibly) x of y in NP titlebar when player is playing
3. touch the track/artist/album area and it launches the context menu
4. (possibly) touch the artwork and it switches to fullscreen artwork

10'UI:
1. relayout according to new guidlines provided by Noah
2. provide a text-only layout of NP
3. provide an art-only layout of NP
4. enable cycling through the different layouts by hitting the NP button on the remote
Comment 3 SVN Bot 2009-10-30 13:56:37 UTC
 == Auto-comment from SVN commit #7983 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7983 ==

Bug: 14665 +1
Description: flip to + button when playlist size is 1
flip to playlist button when playlist size is > 1
Comment 4 SVN Bot 2009-10-30 15:46:25 UTC
 == Auto-comment from SVN commit #7984 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7984 ==

Fixed Bug: 13620 +2
Bug: 14665 +2
Description: On Fab4, touching on the track/artist/album text block raises a context menu
Comment 5 SVN Bot 2009-11-02 14:39:08 UTC
 == Auto-comment from SVN commit #7997 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7997 ==

Bug: 14665
Fixed Bug: 12084
Description: add X of Y to title text for SB Touch skins
make sure X of Y is updated correctly when the playlist changes
Comment 6 SVN Bot 2009-11-03 14:19:41 UTC
 == Auto-comment from SVN commit #8004 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8004 ==

Bug: 14665
Description: Handle the titleStatus text (text-based showBriefly messages that appear in title bar) in as sane a way as possible on Fab4
first line of titleStatus text replaces text in title bar
second line of titleStatus text replaces text on track title label
artist and album labels are suppressed while titleStatus text is being displayed
Comment 7 Ben Klaas 2009-11-03 14:20:30 UTC
updating time spent on NP screen today
Comment 8 Ben Klaas 2009-11-05 07:34:56 UTC
Did a bunch of work last night on toggling through art-only, art+text, text-only views of NP. It's going to be a great feature addition...
Comment 9 SVN Bot 2009-11-05 13:08:42 UTC
 == Auto-comment from SVN commit #8022 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8022 ==

Bug: 14665
Description: implement multiple view Now Playing screen on all skins
On Fab4 10' and 3' UIs, user can cycle through art+text, art only, text only views of the NP screen. This can be done by either touching on the artwork or titlebar or by pressing the Now Playing button on the IR Remote (which I couldn't test because I don't have an IR remote with that button)

Controller/Baby do not have a means of cycling through yet ('go_now_playing' is not an action with any hooks on those devices), but there is a smaller artwork style in both skins that is functional
Comment 10 SVN Bot 2009-11-10 09:15:14 UTC
 == Auto-comment from SVN commit #8039 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8039 ==

Bug: 14665
Description: asset update from Noah, incl. Now Playing ref screens and guidelines for 7.5 release
Comment 11 SVN Bot 2009-11-10 12:39:10 UTC
 == Auto-comment from SVN commit #8043 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8043 ==

Bug: 14665
Description: Add window listener for half a scroll wheel turn to move to next NP view
Make titlebar press launch CM
Make touching artwork _or_ touching text of song toggle to next NP view
Replace space with dot glyph for divider between Now Playing and x of y info
Comment 12 SVN Bot 2009-11-10 12:44:40 UTC
 == Auto-comment from SVN commit #8044 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8044 ==

Bug: 14665
Description: correct bad asset for volume slider
Comment 13 Jim McAtee 2009-11-10 13:38:19 UTC
(In reply to comment #2)
> 
> 3'UI:
> 1. playlist button should switch to + button when playlist size is 1

Can someone explain the logic behind doing this?  In the forums several testers have expressed their confusion over it.  Now that touching the text in the NP screens brings up the context menu, the button seems even more unnecessary.  Inconsistent _and_ unnecessary is generally a bad combination in a user interface.
Comment 14 SVN Bot 2009-11-10 19:57:22 UTC
 == Auto-comment from SVN commit #8047 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8047 ==

Bug: 14665
Description: correction to toolbar asset
Comment 15 SVN Bot 2009-11-10 20:16:11 UTC
 == Auto-comment from SVN commit #8048 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8048 ==

Bug: 14665
Description: text only screen definition for 3' UI, not quite to spec
use skin param to make sure titlebar gets suppressed correctly on art only NP screen
Comment 16 SVN Bot 2009-11-10 20:20:46 UTC
 == Auto-comment from SVN commit #8049 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8049 ==

Bug: 14665
Description: artwork only to design specification, 3' and 10' UIs
Comment 17 SVN Bot 2009-11-10 20:29:25 UTC
 == Auto-comment from SVN commit #8050 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8050 ==

Bug: 14665
Description: use nocturne wallpaper for background in art only screen
Comment 18 SVN Bot 2009-11-10 21:18:20 UTC
 == Auto-comment from SVN commit #8051 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8051 ==

Bug: 14665
Description: Art+Text screen for 10' UI
Progress slider is not perfect because skinning code doesn't play well with slider styles and variable height assets
Comment 19 SVN Bot 2009-11-10 21:19:38 UTC
 == Auto-comment from SVN commit #8052 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8052 ==

Bug: 14665
Description: padding changes to elapsed/remain widgets
Comment 20 SVN Bot 2009-11-10 21:24:40 UTC
 == Auto-comment from SVN commit #8053 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8053 ==

Bug: 14665
Description: further tweaks of 10'UI NP screens
Comment 21 SVN Bot 2009-11-10 22:04:06 UTC
 == Auto-comment from SVN commit #8054 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8054 ==

Bug: 14665
Description: NP screen layouts to spec for 3'UI
Comment 22 Ben Klaas 2009-11-10 22:06:42 UTC
down to misc tweaks with this I think now.

--progress bar slider groups aren't laying out correctly. May have to go to Noah to workaround skinning issues with padded assets, a repeat of what we did for on-screen keyboard.

-- waiting on a text only screen spec for NP in 3' UI. I think what's there is definitely shippable, but I more or less made it up and IANADesigner
Comment 23 SVN Bot 2009-11-10 22:18:24 UTC
 == Auto-comment from SVN commit #8055 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8055 ==

Bug: 14665
Description: layout for no progress bar streams
Comment 24 Ben Klaas 2009-11-13 13:18:59 UTC
I'm calling this done. Cosmetic issues can be raised as separate bugs.
Comment 25 Chris Owens 2010-04-08 17:26:24 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!