<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/107545/">http://git.reviewboard.kde.org/r/107545/</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;">I'm wondering about the looks. We now have 2 inconsistent presence menus - one here and one in contact list; both iconsets are different. I'm not sure that's a good idea. But then again - the icon in the systray is already different.
Andromeda, please add "usability" group in the reviewers, thanks.</pre>
<br />
<p>- Martin</p>
<br />
<p>On December 1st, 2012, 4:59 p.m., Andromeda Galaxy 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 Andromeda Galaxy.</div>
<p style="color: grey;"><i>Updated Dec. 1, 2012, 4:59 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;">The Presence applet will show a Plasma-themed icon on the desktop now. However, the menu items for statuses are still all Oxygen, so the user experience is inconsistent -- if the user clicks on an icon for status, they won't get that icon on the toolbar. The attached diff, if applied to the master, will make it so that those menu items are also Plasma-themed.</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>src/presenceapplet.h <span style="color: grey">(78ccfbd)</span></li>
<li>src/presenceapplet.cpp <span style="color: grey">(291cde4)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107545/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/107545/s/863/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/01/themed-presence-applet_1_400x100.png" style="border: 1px black solid;" alt="The plasma-themed applet" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>