Review Request: Digital Clock applet date formatting

[Marco Martin]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1877/#review2696
-----------------------------------------------------------

Ship it!


seems a good addition

- Marco


On 2009-10-17 22:05:10, John Layt wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1877/
> -----------------------------------------------------------
> 
> (Updated 2009-10-17 22:05:10)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> As discussed on the plasma mailing list.  The Digital Clock applet has problems with l10n and i18n when formatting the date to be displayed, this change fixes these problems and simplifies the config while allowing more options.
> 
> The config GUI has changed from 3 tick-boxes to a single combobox of valid formats, which now includes the system locale formats.  Existing config files are automatically converted to the new config format.
> 
> Comment welcomed on available formats and wording in combobox
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdebase/workspace/plasma/generic/applets/digital-clock/clock.h 1035179 
>   trunk/KDE/kdebase/workspace/plasma/generic/applets/digital-clock/clock.cpp 1035364 
>   trunk/KDE/kdebase/workspace/plasma/generic/applets/digital-clock/clockConfig.ui 1035179 
> 
> Diff: http://reviewboard.kde.org/r/1877/diff
> 
> 
> Testing
> -------
> 
> Tested conversion process for existing configs and new configs.  Tested selecting all available formats in gui.
> 
> 
> Screenshots
> -----------
> 
> Modified config gui
>   http://reviewboard.kde.org/r/1877/s/230/
> Date format combo
>   http://reviewboard.kde.org/r/1877/s/231/
> 
> 
> Thanks,
> 
> John
> 
>