Bug 13146 - (re)design alarm clock app for squeezeplay
: (re)design alarm clock app for squeezeplay
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: UI
: unspecified
: PC Other
: P1 normal (vote)
: 7.4.0
Assigned To: Ben Klaas
: CAT, CX
Depends on:
Blocks: 9570 12164
  Show dependency treegraph
 
Reported: 2009-07-30 13:22 UTC by Weldon Matt
Modified: 2009-10-05 14:35 UTC (History)
6 users (show)

See Also:
Category: Bug


Attachments
setting an alarm flow, controller, 12h (111.83 KB, image/png)
2009-08-29 10:30 UTC, Ben Klaas
Details
setting an alarm flow, controller, 24h (131.00 KB, image/png)
2009-08-29 10:30 UTC, Ben Klaas
Details
setting an alarm flow, baby, 12h (113.21 KB, image/png)
2009-08-29 10:31 UTC, Ben Klaas
Details
setting an alarm flow, baby, 24h (124.17 KB, image/png)
2009-08-29 10:31 UTC, Ben Klaas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Weldon Matt 2009-07-30 13:22:06 UTC
special focus on baby and alarm button.  Involves new time-entry widget.
Comment 1 Weldon Matt 2009-07-30 13:38:13 UTC
*** Bug 9013 has been marked as a duplicate of this bug. ***
Comment 2 ndijulio 2009-07-30 15:02:22 UTC
Can we refer to this widget in a more generic sense, i.e. Multi-character entry.  The same widget can be used for several areas on the ui beyone time...
Comment 3 Weldon Matt 2009-07-30 15:26:15 UTC
This is a time-tracking bug for me/us to deliver a good design for the alarm clock.  Not just the text entry, but the menus and functionality (snooze, multiple alarms etc).  For a more generalized widget for time entry (and other things), we should create/track a separate bug.
Comment 4 Ben Klaas 2009-08-03 07:46:17 UTC
*** Bug 11808 has been marked as a duplicate of this bug. ***
Comment 5 Ben Klaas 2009-08-03 08:34:55 UTC
*** Bug 12944 has been marked as a duplicate of this bug. ***
Comment 6 Weldon Matt 2009-08-11 20:52:20 UTC
Reassessed basic menus, won't recommend any major changes for 7.4.

Noah is doing most of the heavy lifting on the alarm-setting widget in terms of design.  Latest reference screens look good, assuming Tom and/or Ben can build it to spec.

8hrs burned.
Comment 7 Ben Klaas 2009-08-14 08:20:15 UTC
I'm going to pull this bug into my pile and work on the raw implementation.

Advance notice, there are going to be constraints to what we can deliver with this from the UI.

The items in a time input will use the existing resources we have for Menus and items within Menus.

What we can do:
1. restyle the "unselected" columns dynamically
2. have the middle-of-the-screen item always be the one selected
3. do a continuous roll of menu items top-to-bottom (that is, where 12 ends in the hour column, a 1 will show up beneath it.

What we can't do:
1. top-to-bottom fade effect
2. varying heights of menu items

these constraints are imposed by schedule.
Comment 8 Ben Klaas 2009-08-14 11:34:04 UTC
additional UI constraint for 7.4:

5 total items in the menu only on all skins, middle item selected.
Comment 9 James Richardson 2009-08-14 11:34:56 UTC
Lets not forget about 24h mode
Comment 10 SVN Bot 2009-08-14 12:14:28 UTC
 == Auto-comment from SVN commit #7088 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7088 ==

Bug: 13146
Description:
Time input now works, including SC alarms
requires an SC update so alarm set action pops back to the correct window
Comment 11 SVN Bot 2009-08-14 12:16:51 UTC
 == Auto-comment from SVN commit #28188 to the slim repo by bklaas ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28188 ==

Bug: 13146
Description: companion checkin to jive r7088, pop back to correct window after alarm set/update
Comment 12 Seth Schulte 2009-08-26 12:45:39 UTC
Added CAT2 keyword.
Comment 13 SVN Bot 2009-08-26 15:35:54 UTC
 == Auto-comment from SVN commit #28298 to the slim repo by bklaas ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28298 ==

Bug: 13146
Description: use a fairly typical default of 7am for the alarm clock, not midnight
Comment 14 SVN Bot 2009-08-26 15:38:19 UTC
 == Auto-comment from SVN commit #7272 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7272 ==

Bug: 13146
Description: make alarms set between 12:00am and 12:59am work properly
Comment 15 SVN Bot 2009-08-28 10:55:46 UTC
 == Auto-comment from SVN commit #28326 to the slim repo by bklaas ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28326 ==

Bug: 12800
Bug: 13146
Description: support for alarm_state flag in playerstatus response to Squeezeplay
three potential states right now: 'active', 'set', and 'none'
may still add something in that key/val pair for snooze
Comment 16 SVN Bot 2009-08-28 11:07:43 UTC
 == Auto-comment from SVN commit #7308 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7308 ==

Bug: 12800
Bug: 13146
Description: remove playerAlarmSet notification in favor of playerAlarmState
change getAlarmSet method to getAlarmState method
(these are API changes but I feel confident that no one is using them yet; getAlarmState is more appropriate for what is needed)
change clock applet to support new notification method
alarm icon on clock will show unless the alarmState is nil
Comment 17 SVN Bot 2009-08-28 14:29:36 UTC
 == Auto-comment from SVN commit #7312 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7312 ==

Bug: 13146
Fixed Bug: 13750
Bug: 12625
Description: go with pencil line bottom edge on QVGA skin menu items; this is to assist the UI for the "pagination compromise"
most of the UI skin framework for the timeinput widget on both QVGA skins
uppercase AM and PM throughout ampm methods and in UI
Comment 18 SVN Bot 2009-08-28 19:45:28 UTC
 == Auto-comment from SVN commit #7317 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7317 ==

Bug: 13146
Description: final 7.4 UI treatment for setting alarm clock
still need to add 24h clock support
Comment 19 SVN Bot 2009-08-28 20:13:08 UTC
 == Auto-comment from SVN commit #28347 to the slim repo by bklaas ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28347 ==

Bug: 13146
Description: send localized title text 'Add Alarm' in inputSpec for adding alarm
Comment 20 SVN Bot 2009-08-29 10:16:54 UTC
 == Auto-comment from SVN commit #7324 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7324 ==

Fixed Bug: 13146
Description: Full support for setting alarm clock using 24h clock in QVGA skins
Snooze support has a separate bug, so this one can close.
Comment 21 Ben Klaas 2009-08-29 10:29:39 UTC
FYI, Bug 13768 has been opened to track time input on fab4.
Comment 22 Ben Klaas 2009-08-29 10:30:22 UTC
Created attachment 5749 [details]
setting an alarm flow, controller, 12h
Comment 23 Ben Klaas 2009-08-29 10:30:48 UTC
Created attachment 5750 [details]
setting an alarm flow, controller, 24h
Comment 24 Ben Klaas 2009-08-29 10:31:19 UTC
Created attachment 5751 [details]
setting an alarm flow, baby, 12h
Comment 25 Ben Klaas 2009-08-29 10:31:44 UTC
Created attachment 5752 [details]
setting an alarm flow, baby, 24h
Comment 26 James Richardson 2009-10-05 14:35:41 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.