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

Martin Klapetek martin.klapetek at gmail.com
Fri Jul 24 14:33:32 UTC 2015


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

(Updated July 24, 2015, 4:33 p.m.)


Review request for Plasma.


Changes
-------

Upload working diff...rb is having a bad day or smth


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 (updated)
-----

  applets/digital-clock/package/contents/config/main.xml 8b0872b 
  applets/digital-clock/package/contents/ui/DigitalClock.qml 9506078 
  applets/digital-clock/package/contents/ui/configAppearance.qml ead0ad5 

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/20150724/1570c51b/attachment.html>


More information about the Plasma-devel mailing list