Bug 13795 - Alarm UI enhancements
: Alarm UI enhancements
Status: NEW
Product: SqueezePlay
Classification: Unclassified
Component: UI Skin
: unspecified
: PC Windows Vista
: P3 enhancement (vote)
: Future
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-08-31 19:21 UTC by ndijulio
Modified: 2012-02-29 11:04 UTC (History)
2 users (show)

See Also:
Category: Feature


Attachments
Ref artwork - "set" state indication (18.97 KB, image/png)
2009-08-31 19:21 UTC, ndijulio
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ndijulio 2009-08-31 19:21:13 UTC
Created attachment 5760 [details]
Ref artwork - "set" state indication 

1. Need to add 4th state to indicate number has been set by user
-un-selected column, set character = RGB 179. See attached screen comp.

2. Need to center text hort. in column.  Currently pushed too far to right

3. Consider adding back in confirm/done column so that user can preview
selection before submitting.

4. Remove popup "Saving Alarm..." Unnecessary as text is added to menu list

5. Remove popup "All Alarms Disabled" + "All Alarms Enabled". Unnecessary as
text changes with action.

6. User should be prompted to set other options immediately after entering
time. Quick fix would be to push them into the screen ex. Alarm 1: 12:00AM with
options listed below.

Please set priority appropriately.
Comment 1 ndijulio 2009-08-31 19:22:28 UTC
Cc'ing Seth for comment and review.
Comment 2 Seth Schulte 2009-08-31 21:41:34 UTC
This looks very nice.

Probably too much scope creep, but I wonder whether or not we can/should allow users to specify which days of the week a given alarm is valid for.

If they do nothing, then the alarm is "active" and will keep going off until it's deactivated. But they can also choose to go to another settings screen for a given alarm (e.g., Alarm #1 => "set recurrence") and then check off Monday, Tuesday, etc. including "every day". This is particularly useful for weekday vs. weekend. Or, in the case of a student or worker where some days start earlier than others, setting alarms that map to your schedule and then never having to "deactivate Alarm #1, activate Alarm #2" and then reversing that every few days...

Other than that which, like I said, is probably too much for now, I think this looks great.

My only other question is whether or not the "hour" column is inheriting/respecting the "time mode" (or whatever we call it) from Settings/ where the user can specify 24 hour vs. am/pm. If so, and if the user has chosen to display their time using am/pm then we would need a third column (or some other selector?) to specify am or pm.
Comment 3 Ben Klaas 2009-08-31 21:46:35 UTC
1. Not for 7.4
2. Will fix
3. Not for 7.4
4. Update your SC, already fixed this
5. Update your SC, already fixed this
6. Will look into this, but doubt it's doable for 7.4
Comment 4 Ben Klaas 2009-08-31 21:49:17 UTC
Seth, I believe 100% of what you've described is already available.

Alarms can be edited to go on particular days.
Multiple alarms can be set so the weekday vs. weekend alarms (or whatever) are different
24h vs. 12h clock is supported.
Comment 5 SVN Bot 2009-08-31 22:20:51 UTC
 == Auto-comment from SVN commit #7349 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7349 ==

Bug: 13795
Description: better center the numbers horizontally in the columns
Comment 6 Ben Klaas 2009-08-31 22:24:14 UTC
After reflection I don't believe that point #6 should be done and the current behavior is best. After creating the alarm you are popped back to the menu that shows your alarm. If you want to edit it further (particular days, etc.), this is available to you by drilling into this item.

IMO, nothing left to do that's 7.4 P1
Comment 7 ndijulio 2009-09-01 09:26:27 UTC
Thank you Ben.  For 1. Can we add this in ASAP? Of the list this will add the most to usability.

For 6. this is proposed with a new user in mind.  Because of our tree structure a majority of the editable features and settings are hidden unless the user decides to nav into an alarm they just created. I would argue that this is not intuitive upon first usage. I would assume that the menu would take me back into the screen I just came from (time edit or "set time"). None of this is needs to be completed or looked at until Weldon get's back and certainly not for 7.4. 

Is is possible for QA to send out notices to beta testers that they should update/reinstall Squeezebox Server software when a major update has been made?  This way bugs will not be filed that are out of sync with development...
Comment 8 ndijulio 2009-09-02 16:10:59 UTC
Ben- Where does the titlebar text stand?  Currently, the titlebar is blank.  Is this pending direction or just not implemented?

ex. Set Time
Comment 9 Ben Klaas 2009-09-02 17:46:15 UTC
opened bug 13828 for the title bar text in Set Time
Comment 10 Seth Schulte 2009-09-25 17:54:08 UTC
Per Bobbykin/CXR, need to investigate design improvements re visibility of setting advanced alarm features.
Comment 11 Chris Owens 2009-10-21 09:49:38 UTC
moving current p2 bugs to p3 to make room for moving p1.5 bugs to p2
Comment 12 Pat Ransil 2009-10-23 05:11:19 UTC
Administrative move of 7.5 bugs. All P2, P3, P4 being downgraded one level. Will then split P1s.
Comment 13 Pat Ransil 2009-10-23 05:17:25 UTC
Administrative move of 7.5 bugs. All P2, P3, P4 being downgraded one level. Will then split P1s.
Comment 14 Chris Owens 2010-03-08 11:33:06 UTC
Moving lower-priority bugs to next target
Comment 15 Ben Klaas 2010-06-02 10:43:28 UTC
This bug encapsulates what product marketing has asked for with alarms & 7.6