Thanks a lot for this, great job!<div><br></div><div>My review points:</div><div><br></div><div>1) We no longer use "org.freedesktop.Telepathy.Client.KDE.TextUi" as the handler identifier, use "org.freedesktop.Telepathy.Client.KTp.TextUi" instead, same for filetransfer - replace "KDE" with "KTp" (and watch out for the general capitalization, we use KTp, you use KTP sometimes) As for calls (audio/video) - we currently don't really support calls; while it's cool to have it ready, I'd comment it out for the time being.</div>

<div><br></div><div>2) Thanks for following the kdelibs coding style. There are still some minor issues left, like with pointer sign, which should be at the variable side: QObject* parent --> QObject *parent</div><div>

<br></div><div>3) I also agree that keeping the whole model in memory all the time is probably unwanted. The models are pretty quick to load (try opening contact list while connected), so it could probably use lazy loading.</div>

<div><br></div><div>4) Avatar icon vs presence icon - I'd suggest to add the presence icon somewhere else than instead of missing avatar. You could paint is an overlay, but painting pixmaps over pixmaps could be expensive, so I'd suggest to place it before the icons for starting chat, audio etc and maybe visually divide these areas?</div>

<div><br></div><div>Otherwise I'm happy to put it into the 0.4 release.</div><div><div><br></div>--<div><font color="#666666">Martin Klapetek | KDE Developer</font></div><br>
<br><br><div class="gmail_quote">On Sun, Mar 11, 2012 at 23:22, Dan Vratil <span dir="ltr"><<a href="mailto:dan@progdan.cz">dan@progdan.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi,<br>
<br>
I've made a little Krunner plugin for starting chat with IM contacts and I<br>
think it would be nice to have it as part of the KTp framework together with<br>
the plasma applets.<br>
<br>
For now it's in my scratch repo, but I'd like to move it somewhere public,<br>
best to the Telepathy project. I'm of course willing to maintain it there,<br>
there's really not much work since it's really small and simple piece of code.<br>
<br>
Before any action I'd like to get some feedback from you, because I first saw<br>
Telepathy API 48 hours ago, so there will be some place for improvements :)<br>
<br>
Detailed description here:<br>
<br>
<a href="http://www.progdan.cz/2012/03/kde-telepathy-plugin-for-krunner" target="_blank">http://www.progdan.cz/2012/03/kde-telepathy-plugin-for-krunner</a><br>
<br>
and the repo is<br>
<br>
git://<a href="http://anongit.kde.org/scratch/dvratil/ktp-contact-runner.git" target="_blank">anongit.kde.org/scratch/dvratil/ktp-contact-runner.git</a><br>
<br>
Regards,<br>
Dan Vratil<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Dan Vratil<br>
<a href="http://www.progdan.cz" target="_blank">www.progdan.cz</a> | <a href="mailto:dan@progdan.cz">dan@progdan.cz</a> | Jabber: <a href="mailto:progdan@jabber.cz">progdan@jabber.cz</a><br>
Fingerprint: 76C9 2F08 5D0D 6F9E 5AD4 2BFD 3A85 0307 F506 5B61</font></span><br>_______________________________________________<br>
KDE-Telepathy mailing list<br>
<a href="mailto:KDE-Telepathy@kde.org">KDE-Telepathy@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-telepathy" target="_blank">https://mail.kde.org/mailman/listinfo/kde-telepathy</a><br>
<br></blockquote></div><br></div>