Bug 554700

Summary: time settings: NTP must be a checkbox
Product: [openSUSE] openSUSE 11.2 Reporter: Klaus Kämpf <kkaempf>
Component: WebYaSTAssignee: Martin Schmidkunz <mschmidkunz>
Status: RESOLVED FIXED QA Contact: E-mail List <qa-bugs>
Severity: Critical    
Priority: P2 - High CC: jreidinger, jsuchome
Version: RC 1   
Target Milestone: Final   
Hardware: Other   
OS: Other   
Whiteboard:
Found By: Development Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: screen region

Description Klaus Kämpf 2009-11-11 21:12:41 UTC
Created attachment 326957 [details]
screen region

The "Synchronize via Network Time Protocol (NTP)" flag is a radio button, this should be a checkbox instead.

CC mschmidkunz, usability bug
Comment 1 Klaus Kämpf 2009-11-11 21:17:14 UTC
Raising to P1/critical since one cannot un-check a radiobutton
Comment 2 Josef Reidinger 2009-11-12 13:11:19 UTC
It is more to mschidkunz, because I previously have three radio buttons, don't update time, update manually and update via NTP. Then Martin remove first button so you are forced to choose manually time or synchronize via NTP. If there is checkbox what should I do with manually set time (set time manually together with NTP is invalid)?
I have no problem to change it, but I need to know what is intended result.
Comment 3 Josef Reidinger 2009-11-12 13:35:48 UTC
JFYI it will not be fixed in RC2 due to deadline.
Comment 4 Klaus Kämpf 2009-11-12 13:40:45 UTC
If its about alternatives, mark them both clearly by

- putting frames around them
- write a short description (i.e. 'manual', 'automatic') to the frame label
- put the radiobutton left to the frame label
Comment 5 Martin Schmidkunz 2009-11-12 14:19:45 UTC
As discussed with Josef on IRC I will come up with a redesign next week.
Comment 6 Martin Schmidkunz 2009-11-12 17:54:28 UTC
OK, first idea for a redesign:
-------------------------------
[Icon] Time & Date

Timezone [GMT+1:00(Amsterdam,Berlin,Bern, Rom, Stockholm, Wien)|v]
Time [18:45] Date [12/11/2009]
[ ] Adjust time and date automatically

Cancel or [Save]
-------------------------------

Timezone selection would not be split into region and zone anymore.
Usability pros:
* user has to search only once
* conform with user expectation from Windows
* less objects on screen makes orientation easier

Time and Date would in one row and editable, if adjust time and date automatically is not selected (= manual selection).

If "Adjust time and date automatically" is selected, the time and date is greyed out and can`t be edited any more, but the actual time is still displayed.

"Hardware clock is set to UTC" is skipped as I do not know the use case when one should turn it off on purpose (seems it makes more sense to have it turned on by default)

What do you think about that?
Comment 7 Martin Schmidkunz 2009-11-12 22:46:13 UTC
I forgot to mention: when timezone is selected the corresponding time (& date if possible) is shown in the time & date fields.
Comment 8 Martin Schmidkunz 2009-11-12 22:50:33 UTC
*** Bug 546798 has been marked as a duplicate of this bug. ***
Comment 9 Josef Reidinger 2009-11-13 07:50:15 UTC
jiri - is possible to get timezones in such format from yast2-country? (together with information how should I change time in field)
Comment 10 Jiří Suchomel 2009-11-13 08:03:24 UTC
Sorry, YaST currently does not have an information about the time offsets of specific time zones.

Off topic: I think that request to radically change the UI are quite late for this stage of developement...
Comment 11 Josef Reidinger 2009-11-13 08:20:43 UTC
Klaus - What do you think about this change to final release? (It is possible to do, but it is quite big change and need quite lot of testing, as it require to use rails TZinfo structure, which contain some information about timezones and send indetifier to backend (and solve if some incompatibility between rails and yast timezone occur)). Also I expect some problems with daylight saving time.
Comment 12 Klaus Kämpf 2009-11-16 15:31:21 UTC
- no 'big' change before release. Better open another bug for tracking this
- this report is about improving usability, i.e. making manual/automatic time options better visible. RC2 does it reasonably well

-> closing as fixed
-> Jiri/Josef please open a new bug for more 'radical' UI improvements