Tnx : )<br><br>Weer goed werk geleverd. Ik val je weer lastig zodra ik een nieuwe opdracht krijg : )<br><br><br>Mvg,<br>Fatih<br><br><div class="gmail_quote">2009/4/18 Marco Martin <span dir="ltr">&lt;<a href="mailto:notmart@gmail.com">notmart@gmail.com</a>&gt;</span><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im"><br>
-----------------------------------------------------------<br>
This is an automatically generated e-mail. To reply, visit:<br>
</div><a href="http://reviewboard.kde.org/r/603/#review955" target="_blank">http://reviewboard.kde.org/r/603/#review955</a><br>
-----------------------------------------------------------<br>
<br>
<br>
with the premise that i don&#39;t know kopete enough, it seems good to me and seems to work quite good.<br>
tough it opens two questions:<br>
-hiding: should be a function in the api to hide the icon? (i would say if you want to completely remove the icon, delete the knotificationareaitem instance, but seems tricky in this kopete example..)<br>
<br>
-overlays: many apps, like kopete here seems to use composite icons to show status, overlays and things like that, so of course they all have to fallback at the pixmap method.. could be provided some facility in the protocol (i.e setOverlayIcon) or (as i propend now) anyways eveyone will want to do -slightly- different so in the end pixmaps will be used anyways?<br>

<br>
- Marco<br>
<div class="im"><br>
<br>
On 2009-04-17 15:06:26, Davide Bettio wrote:<br>
&gt;<br>
&gt; -----------------------------------------------------------<br>
&gt; This is an automatically generated e-mail. To reply, visit:<br>
&gt; <a href="http://reviewboard.kde.org/r/603/" target="_blank">http://reviewboard.kde.org/r/603/</a><br>
&gt; -----------------------------------------------------------<br>
&gt;<br>
</div>&gt; (Updated 2009-04-17 15:06:26)<br>
<div><div></div><div class="h5">&gt;<br>
&gt;<br>
&gt; Review request for Kopete and Plasma.<br>
&gt;<br>
&gt;<br>
&gt; Summary<br>
&gt; -------<br>
&gt;<br>
&gt; Ported kopete systray icon to KNotificationAreaItem.<br>
&gt;<br>
&gt;<br>
&gt; Diffs<br>
&gt; -----<br>
&gt;<br>
&gt;   /trunk/KDE/kdenetwork/kopete/kopete/CMakeLists.txt 955509<br>
&gt;   /trunk/KDE/kdenetwork/kopete/kopete/kopetewindow.cpp 955509<br>
&gt;   /trunk/KDE/kdenetwork/kopete/kopete/systemtray.h 955509<br>
&gt;   /trunk/KDE/kdenetwork/kopete/kopete/systemtray.cpp 955509<br>
&gt;<br>
&gt; Diff: <a href="http://reviewboard.kde.org/r/603/diff" target="_blank">http://reviewboard.kde.org/r/603/diff</a><br>
&gt;<br>
&gt;<br>
&gt; Testing<br>
&gt; -------<br>
&gt;<br>
&gt;<br>
&gt; Thanks,<br>
&gt;<br>
&gt; Davide<br>
&gt;<br>
&gt;<br>
<br>
_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
</div></div></blockquote></div><br>