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 05:21:13 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115513/
-----------------------------------------------------------

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/015f33aa/attachment.html>


More information about the Plasma-devel mailing list