<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/101989/">http://git.reviewboard.kde.org/r/101989/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin: 1em 0 0 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On July 17th, 2011, 10:58 p.m., <b>Francesco Nwokeka</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding: 0 0 0 1em;">
<br style="display: none;" />
<table bgcolor="#f0f0f0" cellpadding="5" cellspacing="5" style="border: 1px solid #c0c0c0; margin-bottom: 10px">
<tr>
<td><a href="/r/101989/s/205/" style="color: black; font-weight: bold; font-size: 9pt;">contact_list_info.png</a></td>
</tr>
<tr>
<td><a href="http://git.reviewboard.kde.org/r/101989/s/205/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/07/17/contact_list_info_274_291_34_100.png" style="border: 1px black solid;" alt=""></a></td>
</tr>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">maybe make this info bold?</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">How about using icons here?</pre>
<br />
<blockquote style="margin: 1em 0 0 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On July 17th, 2011, 10:58 p.m., <b>Francesco Nwokeka</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding: 0 0 0 1em;">
<br style="display: none;" />
<table bgcolor="#f0f0f0" cellpadding="5" cellspacing="5" style="border: 1px solid #c0c0c0; margin-bottom: 10px">
<tr>
<td><a href="/r/101989/s/205/" style="color: black; font-weight: bold; font-size: 9pt;">contact_list_info.png</a></td>
</tr>
<tr>
<td><a href="http://git.reviewboard.kde.org/r/101989/s/205/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/07/17/contact_list_info_53_289_208_98.png" style="border: 1px black solid;" alt=""></a></td>
</tr>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">we could get these aligned in another patch</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Definitely needs alignment. Also, can we hide the description in a tooltip (perhaps over an question mark icon)?</pre>
<br />
<p>- Martin</p>
<br />
<p>On July 17th, 2011, 10:36 p.m., David Edmundson 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 David Edmundson.</div>
<p style="color: grey;"><i>Updated July 17, 2011, 10:36 p.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;">Contact info dialog, shows information about a user. Accessible via a context menu.
I'm aware it's not the prettiest dialog in the world, but it's better to get something merged which everyone can fix than to have it stuck here doing nothing.</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">(a94fcdaac788e9b5d0c068963112170d83f4f8f4)</span></li>
<li>dialogs/contact-info.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>dialogs/contact-info.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>dialogs/contact-info.ui <span style="color: grey">(PRE-CREATION)</span></li>
<li>main-widget.h <span style="color: grey">(6fd3e8bb0a3999b284a97d5ff6f7f832884411b7)</span></li>
<li>main-widget.cpp <span style="color: grey">(20995f6fd3f038ed3f5046f1ee0cb0a63c1c251c)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101989/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/101989/s/205/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/07/17/contact_list_info_400x100.png" style="border: 1px black solid;" alt="" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>