<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>
<p>On November 16th, 2011, 4:20 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;">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>
</blockquote>
<p>On November 16th, 2011, 5:19 p.m., <b>David Edmundson</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;">Any change this large for 0.2 should have been done ages ago (i.e before hard freeze). So no.</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 just checked Kopete and they are also using their own programatically-created icons plus they ship lots of their own. Someone even suggested doing this in icons request wiki [1] (look for kopete systray icons).
Doing this is imho the best solution, but as you disagree, I propose to create that icon "outside" and ship it as our own, file a bug against Oxygen with this icon attached and hopefully we'll get it included in next Oxygen release. This gives flexibility to users with different icon themes as they can simply replace that installed icon with whatever they want, ie. a theme matching icon.
Are there any objections to this?
[1] - http://techbase.kde.org/Projects/Oxygen/Missing_Icons</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>