Review Request 115513: [digital-clock] Make sure we remove seconds from time string when seconds are disabled
Martin Klapetek
martin.klapetek at gmail.com
Thu Feb 6 17:57:00 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115513/
-----------------------------------------------------------
(Updated Feb. 6, 2014, 5:57 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma.
Bugs: 330813
http://bugs.kde.org/show_bug.cgi?id=330813
Repository: kde-workspace
Description
-------
As https://bugs.kde.org/show_bug.cgi?id=330813 shows, Qt can fallback to "C" locale which returns seconds in "Short Format" time string (afaik it's the only one). This patch makes sure we always remove the seconds part from the format if seconds are not enabled in applet settings.
Diffs
-----
plasma/generic/applets/digital-clock/contents/ui/DigitalClock.qml 5597d2c
Diff: https://git.reviewboard.kde.org/r/115513/diff/
Testing
-------
Works with C locale.
Thanks,
Martin Klapetek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140206/b61e2c87/attachment.html>
More information about the Plasma-devel
mailing list