<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/105514/">http://git.reviewboard.kde.org/r/105514/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On July 11th, 2012, 7:42 p.m., <b>Heiko Tietze</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;">Ship It!</pre>
</blockquote>
<p>On July 11th, 2012, 7:46 p.m., <b>Heiko Tietze</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;">But reconsider the overlay. It works with your phone icon scaled to 8x12px (tested with Gimp).</pre>
</blockquote>
<p>On July 11th, 2012, 8:05 p.m., <b>Dominik Cermak</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;">OK, I'll try the overlay in both variants (over avatar, over presence) and add the screenshots. Where do you think it is better over the presence or the avatar? With the avatar there could be the problem that you can't see it properly because of the avatars colors for example. I could try to make a thin white line around the icon so this should be solved though.
Thanks for your input.</pre>
</blockquote>
<p>On July 11th, 2012, 8:39 p.m., <b>Heiko Tietze</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;">Being mobile or not is part of the contact respectively the avatar, IMHO. I understand it as some kind of a modifier. The reason for my objection was the placement adjacent to presence which could cause trouble when either presence status or personal status will be extended sometime. The overlay can be changed in any direction.</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;">Best solution would be to have special presence icons for mobiles (not overlays) and use them in place of presences. But knowing how long it takes to get new icons...well we could try crafting these ourselves, basically just take the presence icons and cut them in mobile shape. Which I assume will look beyond horrible. I still might try.
As for the presented screenshots (fantastic work Dominik!), for me it's either the original one or the presence icon overlay.</pre>
<br />
<p>- Martin</p>
<br />
<p>On July 11th, 2012, 9:06 p.m., Dominik Cermak 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 and KDE Usability.</div>
<div>By Dominik Cermak.</div>
<p style="color: grey;"><i>Updated July 11, 2012, 9:06 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;">This shows a phone icon left of the presence icon for contacts that are online with a phone.
Currently it only shows the client type of 'phone' (other possible are: bot, console, handheld, pc, web), because I think this is the most important. Also interesting could be 'web'.
The usability group is added for comments/discussion about the place where to put the icon. What do you think about it?</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=302963">302963</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>contact-delegate-compact.cpp <span style="color: grey">(91f3497bac5585d922d1d6c071e3ec6b8796daa1)</span></li>
<li>contact-delegate.cpp <span style="color: grey">(2b8a86398340d9b11e05e9928f9c64f396ce8e00)</span></li>
<li>main-widget.cpp <span style="color: grey">(2baebe3dcbb173ea56586f765b459321bfdad6bd)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105514/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/105514/s/621/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/07/11/clienttype_400x100.png" style="border: 1px black solid;" alt="Full List" /></a>
<a href="http://git.reviewboard.kde.org/r/105514/s/622/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/07/11/clienttype_compact_400x100.png" style="border: 1px black solid;" alt="Compact List" /></a>
<a href="http://git.reviewboard.kde.org/r/105514/s/623/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/07/11/clienttype_compact_left_400x100.png" style="border: 1px black solid;" alt="Version where the icon moved to the left" /></a>
<a href="http://git.reviewboard.kde.org/r/105514/s/624/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/07/11/clienttype_compact_presence_left_400x100.png" style="border: 1px black solid;" alt="Presence moved to the left" /></a>
<a href="http://git.reviewboard.kde.org/r/105514/s/625/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/07/11/clienttype_compact_avatar_overlay_400x100.png" style="border: 1px black solid;" alt="Avatar Overlay" /></a>
<a href="http://git.reviewboard.kde.org/r/105514/s/626/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/07/11/clienttype_compact_presence_overlay_400x100.png" style="border: 1px black solid;" alt="Presence overlay" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>