----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119758/#review64446 -----------------------------------------------------------
applets/digital-clock/package/contents/ui/DigitalClock.qml <https://git.reviewboard.kde.org/r/119758/#comment45034> Do we really want the same text label with the same font? I'd like some screenshots. applets/digital-clock/package/contents/ui/DigitalClock.qml <https://git.reviewboard.kde.org/r/119758/#comment45032> In the size helper we calculate the size of 00:00:00 which is the widest string so that text doesn't jump about. Here you're working out the size of the current day. Which is going to be dynamic. Given this is only calculated once on startup, if you start it in May you'll get it broken if Plasma keeps running till September when the text will no longer fit. applets/digital-clock/package/contents/ui/configAppearance.qml <https://git.reviewboard.kde.org/r/119758/#comment45033> can be just: property string cfg_dateFormat: model[currentIndex]["name"] and remove the onCurrentIndexChanged - David Edmundson On Aug. 13, 2014, 10:51 a.m., Dan Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119758/ > ----------------------------------------------------------- > > (Updated Aug. 13, 2014, 10:51 a.m.) > > > Review request for Plasma. > > > Bugs: 335006 > https://bugs.kde.org/show_bug.cgi?id=335006 > > > Repository: plasma-workspace > > > Description > ------- > > This patch adds option to show date in the DigitalClock applet, something I > was very much missing after switching to Plasma 5 :-) > > Similary to the KDE 4 applet, users can choose from long, short and narrow > date formats. > > > Diffs > ----- > > applets/digital-clock/package/contents/config/main.xml 74ac640 > applets/digital-clock/package/contents/ui/DigitalClock.qml 20babc3 > applets/digital-clock/package/contents/ui/configAppearance.qml ab67e14 > > Diff: https://git.reviewboard.kde.org/r/119758/diff/ > > > Testing > ------- > > > Thanks, > > Dan Vrátil > >
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
