Bug 8880 - DateTime screensaver should show next alarm time
: DateTime screensaver should show next alarm time
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: unspecified
: All All
: -- normal (vote)
: ---
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-07-26 03:50 UTC by Max Spicer
Modified: 2009-09-08 09:28 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
Alarm time displayed on Boom (5.97 KB, image/png)
2008-07-28 01:30 UTC, Michael Herger
Details
using the smaller threeline font (3.05 KB, image/png)
2008-07-28 01:31 UTC, Michael Herger
Details
bottom right - not really right (3.27 KB, image/png)
2008-07-28 07:48 UTC, Michael Herger
Details
top right am/pm (2.95 KB, image/png)
2008-07-28 07:48 UTC, Michael Herger
Details
single line, am/pm (2.24 KB, image/png)
2008-07-28 07:49 UTC, Michael Herger
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Max Spicer 2008-07-26 03:50:11 UTC
The date time screensaver should include the time that the next alarm will go off if that time is within 24 hours.  Non-booms do this, but Boom only displays the bell symbol.  This is tricky due to the restricted width, but it should still be possible.  We'd need to shorten the rest of the information that is displayed - for example the current date display.  This is doable in English as you can use the short date format and fit the alarm time in.  I don't know about other locales.

Without this, it's not easy to check when the next alarm will go off, which is quite important for an alarm clock.

Marking for 7.2 in order to request consideration.  I know Triode has already looked at this, so it could be that it's already been considered not possible.
Comment 1 Adrian Smith 2008-07-26 13:32:38 UTC
Michael - I think you already looked at this and considered it would not fit comfortably?
Comment 2 Michael Herger 2008-07-27 13:54:31 UTC
Yes, I did look at this. And yes, it can work. It just depends on the configuration: if you want the long date displayed, together with am/pm in the alarm's time, then you're a bit out of luck on Boom. Maybe you're right and we should include it with all players, leaving the configuration to the user. Would work fine for me. It happens that our time format is shorter than yours ;-).

There are two screenshots on Campfire:

https://s3.amazonaws.com/campfire/slimdevices/alarm.png?AWSAccessKeyId=1RF809NDDCNB7616HJ02&Expires=1217192291&Signature=eCCOO9Kg0%2BeK1ujl7OPsnFs0RTE%3D

and using the even smaller threelines font:
https://s3.amazonaws.com/campfire/slimdevices/smaller.png?AWSAccessKeyId=1RF809NDDCNB7616HJ02&Expires=1217192286&Signature=KJV%2FbBgLQer9Py%2FDxgiNwOx5HJc%3D

Imagine using "hh:mm pm" on there.
Comment 3 Blackketter Dean 2008-07-27 14:03:31 UTC
michael: those links don't work, can you attach the images to the bug?
Comment 4 Michael Herger 2008-07-28 01:30:44 UTC
Created attachment 3691 [details]
Alarm time displayed on Boom
Comment 5 Michael Herger 2008-07-28 01:31:08 UTC
Created attachment 3692 [details]
using the smaller threeline font
Comment 6 Blackketter Dean 2008-07-28 06:59:48 UTC
I think that the AM/PM will overlap.  Two thoughts:

1.  Display the time, but only if it doesn't overlap with the AM/PM text
2.  Move the icon and time to the _lower_ right corner, where the AM/PM are above the bottom of the screen (to make room for the descenders).  Will it fit there?

Comment 7 Michael Herger 2008-07-28 07:48:17 UTC
Created attachment 3695 [details]
bottom right - not really right
Comment 8 Michael Herger 2008-07-28 07:48:40 UTC
Created attachment 3696 [details]
top right am/pm
Comment 9 Michael Herger 2008-07-28 07:49:20 UTC
Created attachment 3697 [details]
single line, am/pm
Comment 10 Chris Owens 2008-07-28 10:45:25 UTC
Andy suggests that if you're in the daytime screensaver and press the snooze bar that it show the alarm time.
Comment 11 Michael Herger 2008-07-29 03:04:47 UTC
Change 22188 - when in DateTime screensaver mode and there's an alarm within the next 24h, display the next alarm time when hitting the snooze bar.
Comment 12 Max Spicer 2008-07-29 03:11:20 UTC
We'll have to make sure that's documented!
Comment 13 Max Spicer 2008-08-01 14:56:34 UTC
I'm a bit concerned by the string used in line 1 when you hit snooze to see the next alarm time.  It says "Alarm: Enabled", which is a bit odd.  For a start, 'enabled' isn't used anywhere else in the alarm ui - it's always 'on' or 'off'.  Secondly, the state-related word makes me wonder if it's just changed i.e. did pressing snooze enable the alarm?  I think it should just say "Next Alarm".  This makes it much clearer what's going on.  It would be another new string though, I'm afraid.

Another thing, if snooze is pressed and there isn't an alarm, should we say so rather than just doing nothing?
Comment 14 Michael Herger 2008-08-04 02:53:53 UTC
change 22340 - replace "Alarm enabled" with "Next Alarm" when hitting snooze in datetime screensaver mode.

I think it should be obvious enough that there's no alarm set, as there's no bell displayed. IMHO no need to display a message. Feel free to re-open if you don't agree ;-)