----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1877/#review2703 -----------------------------------------------------------
Ship it! +1 hey, since you've been working on the clock, do you have any idea why timezone+date gets a bigger font than the actual time? it's fine if I have only the date and not the timezone... trunk/KDE/kdebase/workspace/plasma/generic/applets/digital-clock/clock.cpp <http://reviewboard.kde.org/r/1877/#comment1986> I'm pretty sure += works with strings too :) - Chani 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 > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel