Review Request 124453: [digital-clock] Provide a simple option for 24h clock

[Martin Klapetek]

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

(Updated Aug. 5, 2015, 12:25 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma and Thomas Pfeiffer.


Changes
-------

Submitted with commit 1fc66aabb2e597fb4b2432a80d4acaa41f305197 by Martin Klapetek to branch master.


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/ui/configAppearance.qml cc1acbe 
  applets/digital-clock/package/contents/ui/DigitalClock.qml 696903a 
  applets/digital-clock/package/contents/config/main.xml 8b0872b 

Diff: https://git.reviewboard.kde.org/r/124453/diff/


Testing
-------

Switching the checkbox on and off works as expected.


Thanks,

Martin Klapetek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150805/e438a8b8/attachment.html>