<div>On Mon, Mar 12, 2012 at 18:46, Dan Vratil <span dir="ltr"><<a href="mailto:dan@progdan.cz">dan@progdan.cz</a>></span> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5">On Monday 12 of March 2012 17:50:57 Martin Klapetek wrote:<br>
> On Mon, Mar 12, 2012 at 15:57, Dan Vratil <<a href="mailto:dan@progdan.cz">dan@progdan.cz</a>> wrote:<br>
> > On Monday 12 of March 2012 11:28:15 Martin Klapetek wrote:><br>
> ><br>
> > 4) Avatar icon vs presence icon - I'd suggest to add the presence icon<br>
> ><br>
> > > somewhere else than instead of missing avatar. You could paint is an<br>
> > > overlay, but painting pixmaps over pixmaps could be expensive, so I'd<br>
> > > suggest to place it before the icons for starting chat, audio etc and<br>
> ><br>
> > maybe<br>
> ><br>
> > > visually divide these areas?<br>
> ><br>
> > As explained by David in a mail above, the presence icon when avatar is<br>
> > missing is there since the first (well, second) commit :)<br>
><br>
> I don't understand - David said the code should use the blue thing instead<br>
> of presence icon, which it in fact does display, but these lines:<br>
><br>
>  if (!iconFile.isEmpty() && QFile::exists(iconFile)) {<br>
>                 match.setIcon(QIcon(iconFile));<br>
>             } else {<br>
>                 match.setIcon(presence.icon());<br>
>             }<br>
><br>
> sets either avatar and if that does not exist, it uses presence icon, which<br>
> is not wanted and should display the blue player (how is that damn thing<br>
> called in english??).<br>
<br>
</div></div>David in the first review mentioned that I should use KTp::Presence to get the<br>
right icon. But I can return the blue damned things :) I like them more<br>
actually :)<br></blockquote><div><br></div><div>Ah, yes, he did. He also stated that "it's a bit lame" to show presence instead of avatar (point nr. 4), which is what I'm talking about ;) Show just avatars or just presences, but don't mix them; it's confusing.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
><br>
> Btw. it crashes for me, BT attached.<br>
<br>
</div>This crash does not seem to be caused by me, isn't this the model crash you<br>
mentioned on IRC?<br></blockquote><div><br></div><div>I don't think so as that one has something to do with TreeNodes.</div><div><br></div><div><div>--</div><div><font color="#666666">Martin Klapetek | KDE Developer</font></div>

</div></div>