<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/109697/">http://git.reviewboard.kde.org/r/109697/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy.</div>
<div>By Daniele E. Domenichelli.</div>
<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;">Add a status message when date changes
BUG: 292041
FIXED-IN: 0.6.0
Make some methods const
Use message sent time instead of received if available
Set time to last content after adding a status message
Set time to the first invalid content message using chat opening time
add{Content,Status}Message should not be Q_SLOTS
AdiumThemeStatusInfo copy constructor should not be explicit
Add a few isXxxx methods AdiumThemeMessageInfo</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;">Tested with text ui (screenshot is actually of a the version with the i18n'ed string) and with log viewer (that gets 2 extra messages before "Previous conversation" and "Next conversation", I'm not sure if this is a bug or a feature)
</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>lib/adium-theme-message-info.h <span style="color: grey">(35ead2368331008deb2b5a8bc303236cd9d92893)</span></li>
<li>lib/adium-theme-message-info.cpp <span style="color: grey">(7726fd2c6be4c59bf420a9f1a0d814466a23b1f0)</span></li>
<li>lib/adium-theme-status-info.h <span style="color: grey">(1cafbe5c413d1cb0dac43dbd00a081b125ac2261)</span></li>
<li>lib/adium-theme-view.h <span style="color: grey">(70618f66ff374c9775391eb0890beb6fddcffd5c)</span></li>
<li>lib/adium-theme-view.cpp <span style="color: grey">(361cd6da826f6d684ddd1a145a6c719848745134)</span></li>
<li>lib/chat-widget.cpp <span style="color: grey">(612fb87774ec7b6a7a1868e33e91cbab8498f67a)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/109697/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<ul>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/03/25/snapshot131.png">Log Viewer</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/03/25/snapshot130_1.png">Text UI</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>