<div>On Mon, Mar 12, 2012 at 15:57, 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="im">On Monday 12 of March 2012 11:28:15 Martin Klapetek wrote:> </div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> 4) Avatar icon vs presence icon - I'd suggest to add the presence icon<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 maybe<br>
> visually divide these areas?<br>
<br>
</div>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></blockquote><div><br></div><div>I don't understand - David said the code should use the blue thing instead of presence icon, which it in fact does display, but these lines:</div>

<div><br></div><div><div> if (!iconFile.isEmpty() && QFile::exists(iconFile)) {</div><div>                match.setIcon(QIcon(iconFile));</div><div>            } else {</div><div>                match.setIcon(presence.icon());</div>

<div>            }</div></div><div><br></div><div>sets either avatar and if that does not exist, it uses presence icon, which is not wanted and should display the blue player (how is that damn thing called in english??).</div>

<div><br></div><div>Btw. it crashes for me, BT attached.</div><div><br></div><div>--<div><font color="#666666">Martin Klapetek | KDE Developer</font></div></div></div>