----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124453/#review82967 -----------------------------------------------------------
I fully agree with you, in both that this should only be a temporary workaround, and on that it is very helpful as such. +1 from me! - Thomas Pfeiffer On July 25, 2015, 9:55 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124453/ > ----------------------------------------------------------- > > (Updated July 25, 2015, 9:55 p.m.) > > > Review request for Plasma and Thomas Pfeiffer. > > > Bugs: 345378 > https://bugs.kde.org/show_bug.cgi?id=345378 > > > Repository: plasma-workspace > > > Description > ------- > > This is _the_ most requested feature for the clock, simple checkbox for 24h > clock. > > In my opinion this is just a temporary workaround and not a very good one > (generally, > code-wise it's awesome :P). There should really be a global config option for > that > but as we now rely on QLocale and QLocale is...bad at this, here's at least > this > small help, which will at least change the clock format on the panel. > > Imo we could really really use something like KLocale again. > > > Diffs > ----- > > applets/digital-clock/package/contents/config/main.xml 8b0872b > applets/digital-clock/package/contents/ui/DigitalClock.qml 9506078 > applets/digital-clock/package/contents/ui/configAppearance.qml ead0ad5 > > Diff: https://git.reviewboard.kde.org/r/124453/diff/ > > > Testing > ------- > > Switching the checkbox on and off works as expected. > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
