<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/108500/">http://git.reviewboard.kde.org/r/108500/</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 b1d9d1df41e8993a79550366e7b5ef4c59942406 by Dan Vrátil to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On January 20th, 2013, 3:10 a.m. UTC, Dan Vrátil wrote:</p>
<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 Dan Vrátil.</div>
<p style="color: grey;"><i>Updated Jan. 20, 2013, 3:10 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;">Based on what Martin suggested in the bug report, I extended the behaviour by also showing an icon when there's an unread message. The icons work as following:
Tab icon:
When there is an unread message in a tab, show "unread message" icon. When there are no unread messages and someone in the chat is typing, show "typing" icon. Otherwise show presence icon. The reasoning is that you are probably more interested in the fact that there is an unread message, than that someone is typing.
Window icon:
When there is an unread message in ANY tab, show "unread message" icon. When there are no unread messages in any tab and someone in CURRENT tab is typing, show "typing" icon. Otherwise show presence icon of current tab. The reasoning again is that you are most interested in the fact that somewhere there is an unread message. The "typing" icon is shown only for current tab, because it might get confusing why the window is indicating "typing" when the current user is not typing.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=311859">311859</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>app/chat-window.h <span style="color: grey">(5a90c91)</span></li>
<li>app/chat-window.cpp <span style="color: grey">(b48f148)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108500/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>