Bug 7263 - Add sleep status icon to NP screen
: Add sleep status icon to NP screen
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: Browser
: unspecified
: All All
: P1 normal (vote)
: 7.4.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-02-21 12:23 UTC by James Richardson
Modified: 2009-10-05 14:33 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
Ref artwork - sleep count-down timer (390.74 KB, image/png)
2009-07-24 18:45 UTC, ndijulio
Details

Note You need to log in before you can comment on or make changes to this bug.
Description James Richardson 2008-02-21 12:23:22 UTC
Currently, if I put the jive to SLEEP manual SETTINGS > SLEEP >, the selected sleep time will be displayed at the top of the menu.

There is no way for the user to know that a sleep has been started, or how much time is left.  Suggest we change the top of the Sleep menu from "Sleeping in 'selected' minutes" to "sleeping in XX minutes" where XX is a count down of time left from when the user selected the sleep amount.

Also, It may be helpful to put a SLEEP icon at the bottom of the "home" and or "now playing" menu.
Comment 1 James Richardson 2008-02-21 12:32:47 UTC
I noticed the JIVE WIKI talks about a "sleep icon" but I don't see one.

http://wiki.slimdevices.com/index.cgi?JiveUI
Comment 2 Blackketter Dean 2008-02-21 15:20:51 UTC
Worth considering for 7.0.1
Comment 3 Ben Klaas 2008-06-18 08:56:12 UTC
7.1 SC change 20894 fixes this

radio buttons replaced with "go" actions and showBriefly messages for user feedback.

time-to-sleep when entering this menu will be the correct amount of minutes remaining until the player sleeps. I will not be implementing a dynamic countdown timer.

I don't know where a sleep icon would go (there really is no space left for it), and we don't have any graphical asset for that. If you feel this is really needed, reopen this bug and assign it to Dean for comment.
Comment 4 James Richardson 2008-06-30 16:08:02 UTC
Dean:

Should we change the wiki to remove reference to this? or add an icon (if possible)
Comment 5 Blackketter Dean 2008-07-17 09:31:18 UTC
I updated the wiki to remove the sleep icon:

http://wiki.slimdevices.com/index.php/ControllerUI

Let's consider this for the UI revamp in 7.4
Comment 6 Blackketter Dean 2008-12-01 21:30:01 UTC
Reassigning...
Comment 7 Mike 2009-01-24 17:22:14 UTC
Please also see forum posting http://forums.slimdevices.com/showthread.php?t=58766 for additional options/settings.
Comment 8 Blackketter Dean 2009-01-24 17:48:53 UTC
Noah, we'll need sleep and alarm indicators on the now playing screen for all SqueezePlay-based UIs for 7.4.
Comment 9 Blackketter Dean 2009-07-22 08:39:01 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 single original product.
Comment 10 ndijulio 2009-07-22 09:28:21 UTC
Matt-

The proposal would be to add a sleep icon indicator to the Now Playing screen.  This is something we had discussed independently.  Thoughts?  It would most likely go inside the content menu (floating) or the status bar if there is room.

We have an icon in svn. for fab4, however, not for the QVGAskins.
Comment 11 Weldon Matt 2009-07-22 17:03:58 UTC
Noah, do you have a reference screen (even a rough one) for how this might look?  Having trouble envisioning where it would fit, esp. on fab4.
Comment 12 ndijulio 2009-07-23 09:10:04 UTC
Could we add a static, status indicator icon to the Now Playing and look into including a elapsed time to shutdown in settings perhaps?  I would strongly recommend including any form of count-down timer in a settings menu.  My concern is two-fold:

1. Placement on the Now Playing screen for Controller = space is limited.

2. On the Now Playing screen with time (clock) and song progress (elapsed time) adding another countdown timer could get confusing quickly:)
Comment 13 Mike 2009-07-23 09:13:54 UTC
Do we need the count down timer showing, why not just a small animated icon.?
Then pressing the sleep button would show the timer in question.
Comment 14 Weldon Matt 2009-07-23 11:58:18 UTC
FWIW, the latest version of the sleep control in the settings menu does now show you the time remaining.  I don't think it worked that way when this bug was originally filed.
Comment 15 ndijulio 2009-07-24 14:08:49 UTC
Ben - over to you.  The "sleep mode" indicator icon has been added to svn.  Guidelines + ref. screens are added as well.  In addition, an alarm mode icon has been added for all platforms.

The second part of this bug: count down timer has already been implemented by you.  As discussed over the phone we might want to revisit the exact implementation post 7.4.

Matt- Please review what Ben has added to the Settings sleep menu to indicate time remaining until sleep.
Comment 16 Ben Klaas 2009-07-24 14:14:06 UTC
I'm viewing this as a necessary UI feature, so removing the enhancement severity
Comment 17 ndijulio 2009-07-24 18:45:55 UTC
Created attachment 5523 [details]
Ref artwork - sleep count-down timer

Version #2 of the attached is the preferred direction. Not sure why we need to have an entire sub menu for this?
Comment 18 Richard Titmuss 2009-07-27 01:13:10 UTC
Reset priority before triage.
Comment 19 SVN Bot 2009-08-09 10:41:52 UTC
 == Auto-comment from SVN commit #6978 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=6978 ==

Fixed Bug: 7263
Description: add sleep icon to NP screen (baby and controller skins)
Comment 20 James Richardson 2009-10-05 14:33:22 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.