Review Request: ClockApplet : show seconds in the tooltip

Iamluc luc at vieillescazes.net
Sat Jan 29 15:15:29 CET 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6336/
-----------------------------------------------------------

(Updated Jan. 29, 2011, 2:15 p.m.)


Review request for Plasma.


Changes
-------

Hi,

New version, with Marco and Aaron recommandations.

Use a QObject based class (ClockTimer) and connect a signal between ClockTimer and the ClockApplet.


Summary
-------

Hi,

When using a clock applet, seconds are often not visibles. But sometimes you need them.
This patch shows them in the tooltip of the applet.

Analog-clock has been updated to refresh the tooltip every seconds.
If this change is accepted, I could change digital-clock

Thanks !


Diffs (updated)
-----

  /trunk/KDE/kdebase/workspace/libs/plasmaclock/clockapplet.h 1217808 
  /trunk/KDE/kdebase/workspace/libs/plasmaclock/clockapplet.cpp 1217808 

Diff: http://svn.reviewboard.kde.org/r/6336/diff


Testing
-------

It Works on plasmoidviewer and with a real session


Thanks,

Iamluc

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110129/c433b96d/attachment.htm 


More information about the Plasma-devel mailing list