Review Request: [MOCKUP] Clock: remove "Use local time zone"

[Aaron Seigo]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.vidsolbach.de/r/186/#review191
-----------------------------------------------------------

Ship it!


i like it much better with the box on top.. nice work.

now i suppose i'll need to add checkboxes to the time zone widget ;)

- Aaron


On 2008-09-10 14:27:35, Rafa? Mi?ecki wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.vidsolbach.de/r/186/
> -----------------------------------------------------------
> 
> (Updated 2008-09-10 14:27:35)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Today we agreed with Aaron and Anne-Marie that we need some cleaning in time zone(s) selection (specially choices including local time zone).
> My idea is to remove "Use local time zone", keep KTimeZoneWidget and QComboBox always enabled and put "Local" at top of QComboBox (no matter what is selected in KTimeZoneWidget). If user additionaly selects some other time zones in KTimeZoneWidget, then theses time zones are avaiable to choose in "Default clocks to" (QComboBox). Of course "Local" is then available as well.
> What do you think about such a solution?
> 
> 
> EDIT:
> 
> -----
> 
> <aseigo> Zajec: as for the ui, i'd put the combobox on the top now .. so that it communicates clearly to the user that they don't need to select something from the listing
> 
> I added next mockup with this suggestion applied.
> 
> -----
> 
> From: Celeste Lyn Paul
> Date: 2008/9/10
> Subject: Re: Review Request: [MOCKUP] Clock: remove "Use local time zone"
> To: plasma-devel at kde.org
> 
> Checkboxes are much easier and intuitive for making multiple selections than
> the current selection method.  As it is, theres no way to know you can make
> multiple selections.  Checkboxes provide that "multi" affordance.
> 
> I guess this is comment about whole KTimeZoneWidget? If so it doesn't really have anything to this idea. However this may be worth of discussion. Maybe you could open some bug report to discuss this?
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/applets/libplasmaclock/clockapplet.cpp
>   /trunk/KDE/kdebase/workspace/plasma/applets/libplasmaclock/timezonesConfig.ui
> 
> Diff: http://reviewboard.vidsolbach.de/r/186/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> New version of configuration dialog
>   http://reviewboard.vidsolbach.de/r/186/s/60/
> Next version of mockup
>   http://reviewboard.vidsolbach.de/r/186/s/61/
> 
> 
> Thanks,
> 
> Rafa?
> 
>