<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/101389/">http://git.reviewboard.kde.org/r/101389/</a>
</td>
</tr>
</table>
<br />
<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 May 19, 2011, 12:04 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Added icon for Hidden presence, solved some spacing issues in delegate and made the popup html code easier and reorganised (see the screenshot).</pre>
</td>
</tr>
</table>
<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 patch adds a switch for an alt. delegate, which is more compact than the original one. Also it refactors the delegate code - now there's a base class, which takes care of the tooltip and of the shared painting stuff (groups etc), but can be easily overriden by custom painting. This class can be used for other possible delegates.</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;">Switched the delegate few times, all is ok.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>CMakeLists.txt <span style="color: grey">(919df77)</span></li>
<li>abstract-contact-delegate.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>abstract-contact-delegate.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>contact-delegate-compact.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>contact-delegate-compact.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>contact-delegate.h <span style="color: grey">(46fea76)</span></li>
<li>contact-delegate.cpp <span style="color: grey">(209e715)</span></li>
<li>contact-overlays.h <span style="color: grey">(c858b73)</span></li>
<li>main-widget.h <span style="color: grey">(7a5e417)</span></li>
<li>main-widget.cpp <span style="color: grey">(20e8003)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101389/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/101389/s/169/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/05/19/snapshot12_400x100.png" style="border: 1px black solid;" alt="New delegate screenshot" /></a>
<a href="http://git.reviewboard.kde.org/r/101389/s/170/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/05/19/snapshot13_400x100.png" style="border: 1px black solid;" alt="New contact tooltip" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>