<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/103063/">http://git.reviewboard.kde.org/r/103063/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On November 7th, 2011, 7:32 p.m., <b>Daniele Elmo Domenichelli</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Looks nice, but I'm a bit worried about what could happen with other icon themes... did you try any other?
Perhaps it should be a real icon "view-sort-im-status" or something similar?</pre>
</blockquote>
<p>On November 7th, 2011, 7:38 p.m., <b>Martin Klapetek</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Yes, our own icon would be super nice. With other themes - I'm not really sure, but one thing will definitely stay clear to the user - the top icon will be online and the second will be away. And these will be the same as everywhere else (presence drop-down, presence plasmoid etc) so it should get the user some clue. The background icon can change significantly, although it's such a specific icon that I think it would stay mostly similar. That and plus the tooltip should work. At least before we get our own icon.</pre>
</blockquote>
<p>On November 8th, 2011, 5:16 p.m., <b>Daniele Elmo Domenichelli</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<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 think it could be quite confusing if, for example, the view-sort-ascending icon has the arrow on the left side instead of on the right side...</pre>
</blockquote>
</blockquote>
<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've sent an email to kde-artists for their opinion few days ago and received no response as of now. Since the 0.2 release is very near, let's decide this ourselves.
Do we ship it for 0.2 or not?</pre>
<br />
<p>- Martin</p>
<br />
<p>On November 7th, 2011, 4:40 p.m., Martin Klapetek 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 Martin Klapetek.</div>
<p style="color: grey;"><i>Updated Nov. 7, 2011, 4:40 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;">Sometime ago someone changed the icon for sorting by presence to "user-online". The icon didn't really fit in the toolbar and didn't make too much sense. Since there is no good icon, I've "created" my own by taking the "view-sort-ascending" and painting two small overlays over it. See the screenshot.</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>main-widget.cpp <span style="color: grey">(05cf410)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103063/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/103063/s/326/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/11/07/cl-icon-fix_400x100.png" style="border: 1px black solid;" alt="New toolbar icon" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>