Hi !<br><br>Just, a small reminder about that patch. :-)<br><br>I read in an another thread that you want to change the calendar popup and the tootip, so maybe that patch is not necessary anymore.<br>But please in the new solution, keep the seconds visibles :-)<br>

<br>Thanks.<br><br>Luc.<br><br><br><div class="gmail_quote">2011/2/5 Iamluc <span dir="ltr">&lt;<a href="mailto:luc@vieillescazes.net">luc@vieillescazes.net</a>&gt;</span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">





 <div>
  <div style="font-family: Verdana,Arial,Helvetica,Sans-Serif;"><div class="im">
   <table style="border: 1px solid rgb(201, 195, 153);" width="100%" bgcolor="#f9f3c9" cellpadding="8">
    <tbody><tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://svn.reviewboard.kde.org/r/6336/" target="_blank">http://svn.reviewboard.kde.org/r/6336/</a>
     </td>
    </tr>
   </tbody></table>
   <br>


</div><table style="background-repeat: repeat-x; border: 1px solid black;" width="100%" bgcolor="#fefadf" cellpadding="8" cellspacing="0">
 <tbody><tr>
  <td><div class="im">

<div>Review request for Plasma.</div>
<div>By Iamluc.</div>


</div><p style="color: grey;"><i>Updated Feb. 5, 2011, 2:26 p.m.</i></p>



<h1 style="color: rgb(87, 80, 18); font-size: 10pt; margin-top: 1.5em;">Changes</h1>
<table style="border: 1px solid rgb(184, 181, 160);" width="100%" bgcolor="#ffffff" cellpadding="10" cellspacing="0">
 <tbody><tr>
  <td>
   <pre style="margin: 0pt; padding: 0pt; white-space: pre-wrap; word-wrap: break-word;">Hi,

I followed Aaron comments :

* rename ClockTimer to something clearer -&gt; ClockToolTip?
---&gt; Done

* have ClockToolTip subclass both QObject and Plasma::ToolTipContent
--&gt; Done

* move updateToolTip into ClockToolTip, renamed as dataUpdated(const QString &amp;source, const Plasma::DataEngine &amp;data) ;)
--&gt; Done, and move function addTzToTipText() too.

* connect to the dataengine on creation of ClockToolTip
--&gt; Done

* create a ClockToolTip object when toolTipAboutToShow is called
* delete it in toolTipHidden
--&gt; Both done

* pass in the ClockApplet as the parent to ClockToolTip and then in ClockToolTip::dataUpdated, set the mainText and then call Plasma::ToolTipManager::self()-&gt;setContent(clockApplet, this)
--&gt; Done
</pre>
  </td>
 </tr>
</tbody></table><div class="im">


<h1 style="color: rgb(87, 80, 18); font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table style="border: 1px solid rgb(184, 181, 160);" width="100%" bgcolor="#ffffff" cellpadding="10" cellspacing="0">
 <tbody><tr>
  <td>
   <pre style="margin: 0pt; padding: 0pt; white-space: pre-wrap; word-wrap: break-word;">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 !</pre>
  </td>
 </tr>
</tbody></table>


<h1 style="color: rgb(87, 80, 18); font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table style="border: 1px solid rgb(184, 181, 160);" width="100%" bgcolor="#ffffff" cellpadding="10" cellspacing="0">
 <tbody><tr>
  <td>
   <pre style="margin: 0pt; padding: 0pt; white-space: pre-wrap; word-wrap: break-word;">It Works on plasmoidviewer and with a real session</pre>
  </td>
 </tr>
</tbody></table>




<h1 style="color: rgb(87, 80, 18); font-size: 10pt; margin-top: 1.5em;">Diffs (updated)</h1>
</div><ul style="margin-left: 3em; padding-left: 0pt;">

 <li>/trunk/KDE/kdebase/workspace/libs/plasmaclock/CMakeLists.txt <span style="color: grey;">(1217808)</span></li><div class="im">

 <li>/trunk/KDE/kdebase/workspace/libs/plasmaclock/clockapplet.h <span style="color: grey;">(1217808)</span></li>

 <li>/trunk/KDE/kdebase/workspace/libs/plasmaclock/clockapplet.cpp <span style="color: grey;">(1217808)</span></li>

 </div><li>/trunk/KDE/kdebase/workspace/libs/plasmaclock/clocktooltip.h <span style="color: grey;">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdebase/workspace/libs/plasmaclock/clocktooltip.cpp <span style="color: grey;">(PRE-CREATION)</span></li>

</ul>

<p><a href="http://svn.reviewboard.kde.org/r/6336/diff/" style="margin-left: 3em;" target="_blank">View Diff</a></p>




  </td>
 </tr>
</tbody></table>




  </div>
 </div>


</blockquote></div><br>