<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/103297/">http://git.reviewboard.kde.org/r/103297/</a>
     </td>
    </tr>
   </table>
   <br />





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 3bf6a533141f6f4a0f68540798f61ad4c9d84258 by Martin Klapetek to branch master.</pre>
 <br />







<p>- Commit</p>


<br />
<p>On December 2nd, 2011, 11:21 a.m., Martin Klapetek wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Telepathy.</div>
<div>By Martin Klapetek.</div>


<p style="color: grey;"><i>Updated Dec. 2, 2011, 11:21 a.m.</i></p>






<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This is the result of Google Code-in student Geoffry Song's work, based on David's code (which was stolen from system settings). The tooltips works perfectly, the code is great quality as well, though as it is rather big diff, I'd like to have more eyes on it.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I've been using it for two days now and all hail to Geoffry, who did really awesome job.</pre>
  </td>
 </tr>
</table>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>CMakeLists.txt <span style="color: grey">(68dc90c)</span></li>

 <li>abstract-contact-delegate.cpp <span style="color: grey">(2bd0f55)</span></li>

 <li>main-widget.cpp <span style="color: grey">(7f0adf1)</span></li>

 <li>tooltips/contacttooltip.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tooltips/contacttooltip.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tooltips/contacttooltip.ui <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tooltips/ktooltip.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tooltips/ktooltip.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tooltips/ktooltipwindow.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tooltips/ktooltipwindow_p.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tooltips/tooltipmanager.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tooltips/tooltipmanager.cpp <span style="color: grey">(PRE-CREATION)</span></li>

</ul>

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




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








  </div>
 </body>
</html>