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

(Updated Aug. 18, 2014, 5:34 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Repository: kdeplasma-addons


Description
-------

This ports the infamous Fuzzy Clock to Plasma 5.

The code is derived from digital clock with the fuzzy logic derived from the 
original implementation.

I had to substitute %1 by $1 in the i18n strings (and replace them back), 
otherwise I got "I18N_EXCESSIVE_ARGUMENTS" thingies appended to my fuzzy string.

Missing is the ability to show the time zone (really needed here?), and the 
"Percent of taskbar size" slider because that layouting logic copied from 
digital clock is beyond me :)
Also missing is the "Configure time format" (why's there no "Configure Date and 
Time" in digital clock?) because fuzzy clock doesn't really adhere to the 
locale anyway and I didn't want to yet again duplicate that ProcessRunner 
plugin which seems to have been copied all over the place already.


Diffs
-----

  applets/fuzzy-clock/plasma-clock-fuzzy.desktop 5f6d30b 
  applets/fuzzy-clock/package/contents/config/main.xml PRE-CREATION 
  applets/fuzzy-clock/package/contents/ui/FuzzyClock.qml PRE-CREATION 
  applets/fuzzy-clock/package/contents/ui/configAppearance.qml PRE-CREATION 
  applets/fuzzy-clock/package/contents/ui/main.qml PRE-CREATION 
  applets/fuzzy-clock/package/metadata.desktop PRE-CREATION 
  applets/fuzzy-clock/CMakeLists.txt 1068150 
  applets/fuzzy-clock/Messages.sh c8c9f06 
  applets/fuzzy-clock/fuzzyClock.h 9bf5c4e 
  applets/fuzzy-clock/fuzzyClock.cpp 2cd189d 
  applets/fuzzy-clock/fuzzyClockConfig.ui 15cc658 
  applets/fuzzy-clock/package/contents/config/config.qml PRE-CREATION 

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


Testing
-------

I've been running it since yesterday evening and didn't notice unusual 
behavior, except that due to the update interval of 30s it doesn't update right 
away when session is resumed from Suspend (but I guess this is a Plasma issue?).

Also it tends to cut off in a vertical panel due to its (sane) minimum font 
size (smallest theme font) being larger than in the original implementation 
where it used to get super tiny then.


File Attachments
----------------

In horizontal panel
  
https://git.reviewboard.kde.org/media/uploaded/files/2014/08/12/35e0fea1-8a28-4ddb-9a81-9112bb85eab5__fuzzyinapanel.png
On the Desktop
  
https://git.reviewboard.kde.org/media/uploaded/files/2014/08/12/a246f3b3-bb12-4756-8069-594d25bd39f6__fuzzyonthedesktop.png
Configuration UI
  
https://git.reviewboard.kde.org/media/uploaded/files/2014/08/12/a144ea07-e18a-4a90-965a-3db51d91fb5d__fuzzyconfig.png


Thanks,

Kai Uwe Broulik

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to