----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127102/#review93561 -----------------------------------------------------------
I've tested it extensively and it works great. Thanks a lot! There's one more issue - http://paste.opensuse.org/view/raw/f8ba5d0d - the same font size should also be applied to the left date label (there are; the bottom should stay as is). Do you think you could include it as part of this patch? applets/digital-clock/package/contents/ui/DigitalClock.qml (line 554) <https://git.reviewboard.kde.org/r/127102/#comment63784> spaces around operators applets/digital-clock/package/contents/ui/DigitalClock.qml (lines 564 - 567) <https://git.reviewboard.kde.org/r/127102/#comment63785> Can't we just compare "A" and "P" width's and use that? Would spare creating two Date objects and two calls to Qt.formatTime - Martin Klapetek On March 8, 2016, 6:53 p.m., Daniel Faust wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127102/ > ----------------------------------------------------------- > > (Updated March 8, 2016, 6:53 p.m.) > > > Review request for kde-workspace and Plasma. > > > Bugs: 347724 > https://bugs.kde.org/show_bug.cgi?id=347724 > > > Repository: plasma-workspace > > > Description > ------- > > Currently the width of the date label is not fixed but changes depending on > the text. This causes the entire applet to change its width (if the time is > the widest displayed item). This in turn can cause all other applets in the > same panel to move whenever the displayed time changes. > > This patch uses FontMetrics to iterate over all possible time strings (with > different width) and chooses the widest of them as reference for the fixed > width of the time label. > > This way the width of the applet stays the same (unless the date is displayed > and changes). The text remains centered though, which means that it can still > move within the applet when the time changes. > > > Diffs > ----- > > applets/digital-clock/package/contents/ui/DigitalClock.qml 95bb071 > > Diff: https://git.reviewboard.kde.org/r/127102/diff/ > > > Testing > ------- > > Works with horizontal and vertical panel. > Also displaying different combinations of "seconds", "date" and "timezone" > works. > > > Thanks, > > Daniel Faust > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
