
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd">

<html><head><meta name="qrichtext" content="1" /><style type="text/css">

p, li { white-space: pre-wrap; }

</style></head><body style=" font-family:'DejaVu Sans'; font-size:10pt; font-weight:400; font-style:normal;">

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Maybe the last activity device shall be reflected... activity may mean change of status... or sending messages.</p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Alin</p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">On Thu 12 Jul 2012 13:42:58 Dominik Cermak wrote:<br /></p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<table border="0" style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px;" width="100%" cellspacing="2" cellpadding="8" bgcolor="#f9f3c9">

<tr>

<td>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';">This is an automatically generated e-mail. To reply, visit: </span><a href="http://git.reviewboard.kde.org/r/105514/"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; text-decoration: underline; color:#0057ae;">http://git.reviewboard.kde.org/r/105514/</span></a><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span></p></td></tr></table>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"><br /></span></p>

<p style=" margin-top:12px; margin-bottom:12px; margin-left:58px; margin-right:80px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';">On July 12th, 2012, noon, </span><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; font-weight:600;">George Kiagiadakis</span><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> wrote: </span></p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<table border="0" style=" margin-top:0px; margin-bottom:0px; margin-left:76px; margin-right:120px;" width="100%" cellspacing="2" cellpadding="0" bgcolor="#ffffff"><thead>

<tr>

<td colspan="4" bgcolor="#f0f0f0" style=" padding-left:8; padding-right:8; padding-top:4; padding-bottom:4;">

<p align="center" style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><a href="http://git.reviewboard.kde.org/r/105514/diff/1/?file=72059#file72059line105"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; font-size:9pt; font-weight:600; text-decoration: underline; color:#000000;">contact-delegate.cpp</span></a><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; font-size:9pt; font-weight:600;"> </span><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; font-size:9pt;">(Diff revision 1) </span></p></td></tr></thead>

<tr>

<td colspan="4">

<pre style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Courier New,courier'; font-size:8pt;">void ContactDelegate::paintContact(QPainter *painter, const QStyleOptionViewItem &option, const QModelIndex &index) const</span></pre></td></tr>

<tr>

<td bgcolor="#b1ebb0"></td>

<td width="50%" bgcolor="#c5ffc4"></td>

<td bgcolor="#b1ebb0">

<p align="right" style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; font-size:small; font-weight:600;">105</span></p></td>

<td width="50%" bgcolor="#c5ffc4">

<pre style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Courier New,courier'; font-size:8pt;">    if (index.data(AccountsModel::ClientTypesRole).toStringList().contains(QLatin1String("phone"))) {</span></pre></td></tr></table>

<p style=" margin-top:12px; margin-bottom:0px; margin-left:76px; margin-right:120px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Courier New,courier';">I'm not sure this is correct.</span></p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:76px; margin-right:120px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Courier New,courier';">This list will contain all the client types for the contact's current presence. For example, if the contact is "available" on the pc and "away" on the phone, this list will contain only "pc" and the presence of the contact will be "available". Vice versa, if the user is "available" on the phone and "away" on the pc, this list will contain only "phone" and the presence will be "available". Everything is fine up to this point. If you try to talk to this person, XMPP should normally choose to notify the most available resource of this contact, which would be the pc in the first case and the phone in the second case.</span></p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<p style=" margin-top:0px; margin-bottom:12px; margin-left:76px; margin-right:120px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Courier New,courier';">However, if a contact is "available" on both the pc and the phone, this list will contain both "pc" and "phone". Therefore, if you try to talk to this person, she should be notified on both devices. In this case, imho, the phone icon should not be there, as there is at least one other client that is not a phone that will be notified.</span><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span></p>

<p style=" margin-top:12px; margin-bottom:12px; margin-left:58px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Courier New,courier';">This is not how it works, if a contact is "available" on both the pc and the phone the list will contain only "pc" or only "phone". Which one is shown depends on which changed to this status last. So if I'm "available" with my pc, then connect with my phone (also "available") clientTypes would contain only "phone" and vice versa. It works in the same way as the presence messages.</span></p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"><br /></span></p>

<p style=" margin-top:12px; margin-bottom:12px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';">- Dominik <br /></span></p>

<p style=" margin-top:12px; margin-bottom:12px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';">On July 11th, 2012, 9:06 p.m., Dominik Cermak wrote: </span></p>

<table border="0" style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px;" width="100%" cellspacing="0" cellpadding="8" bgcolor="#fefadf">

<tr>

<td background="http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png">

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';">Review request for Telepathy and KDE Usability.</span></p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';">By Dominik Cermak. </span></p>

<p style=" margin-top:12px; margin-bottom:12px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; font-style:italic; color:#808080;">Updated July 11, 2012, 9:06 p.m.</span><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span></p>

<p style=" margin-top:27px; margin-bottom:12px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; font-size:xx-large; font-weight:600; color:#575012;">Description </span></p>

<table border="0" style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px;" width="100%" cellspacing="0" cellpadding="10" bgcolor="#ffffff">

<tr>

<td background="http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png">

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Courier New,courier';">This shows a phone icon left of the presence icon for contacts that are online with a phone.</span></p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Courier New,courier';">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'.</span></p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Courier New,courier';">The usability group is added for comments/discussion about the place where to put the icon. What do you think about it?</span><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span></p></td></tr></table>

<p style=" margin-top:27px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; font-weight:600; color:#575012;">Bugs: </span><a href="http://bugs.kde.org/show_bug.cgi?id=302963"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; text-decoration: underline; color:#0057ae;">302963</span></a><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span></p>

<p style=" margin-top:27px; margin-bottom:12px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; font-size:xx-large; font-weight:600; color:#575012;">Diffs </span></p>

<ul style="margin-top: 0px; margin-bottom: 0px; margin-right: 0px; -qt-list-indent: 1;"><li style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';" style=" margin-top:12px; margin-bottom:0px; margin-left:54px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">contact-delegate-compact.cpp <span style=" color:#808080;">(91f3497bac5585d922d1d6c071e3ec6b8796daa1)</span> </li>

<li style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';" style=" margin-top:0px; margin-bottom:0px; margin-left:54px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">contact-delegate.cpp <span style=" color:#808080;">(2b8a86398340d9b11e05e9928f9c64f396ce8e00)</span> </li>

<li style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';" style=" margin-top:0px; margin-bottom:12px; margin-left:54px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">main-widget.cpp <span style=" color:#808080;">(2baebe3dcbb173ea56586f765b459321bfdad6bd)</span> </li></ul>

<p style=" margin-top:12px; margin-bottom:12px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><a href="http://git.reviewboard.kde.org/r/105514/diff/"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; text-decoration: underline; color:#0057ae;">View Diff</span></a><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span></p>

<p style=" margin-top:27px; margin-bottom:12px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif'; font-size:xx-large; font-weight:600; color:#575012;">Screenshots </span></p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><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" /></a><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span><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" /></a><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span><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" /></a><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span><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" /></a><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span><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" /></a><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span><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" /></a><span style=" font-family:'Verdana,Arial,Helvetica,Sans-Serif';"> </span></p></td></tr></table>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /><br /></p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">-- </p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Without Questions there are no Answers!</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">______________________________________________________________________</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Alin Marin ELENA</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Advanced Molecular Simulation Research Laboratory</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">School of Physics, University College Dublin</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">----</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Ardionsambl� M�il�neach Saotharlann Taighde</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Scoil na Fisice, An Col�iste Ollscoile, Baile �tha Cliath</p>

<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">-----------------------------------------------------------------------------------</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">http://alin.elenaworld.net</p>

<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">______________________________________________________________________</p></body></html>