<br><br><div class="gmail_quote">On Tue, May 24, 2011 at 2:35 PM, David Edmundson <span dir="ltr"><<a href="mailto:david@davidedmundson.co.uk">david@davidedmundson.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Forwarded a message from nwoki about the presence plasmoid.<div><br></div><div>Nwoki: You may as well stick to the main ML, unless it's something particularly secret.<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>
From: <b class="gmail_sendername">Francesco Nwokeka</b> <span dir="ltr"><<a href="mailto:francesco.nwokeka@gmail.com" target="_blank">francesco.nwokeka@gmail.com</a>></span><br>Date: Tue, May 24, 2011 at 2:05 PM<br>
Subject: Presence Plasmoid REMAKE<br>
To: David Edmundson <<a href="mailto:david@davidedmundson.co.uk" target="_blank">david@davidedmundson.co.uk</a>><br>Cc: <a href="mailto:drf@kde.org" target="_blank">drf@kde.org</a><br><br><br>Hi mentors! [and other people - ed]<div>
<div></div><div class="h5"><br>
The presence plasmoid is almost ready. It can now set the presence status.<br>
Today i'll be fixing a bug in the dataengine and then I'll be working on the "CMakeLists.txt" file for<br>
the plasmoid installation.<br>
If you want to follow it you can find my repo here[1].<br>
Please send feedback especially with negative observations if possible so that I can make it better<br>
where needed.<br>
I'll be online all day and night on irc so you can catch me whenever you want.<br>
<br>
<br>
Francesco<br>
<br>
<br>
[1] <a href="http://quickgit.kde.org/?p=scratch%2Fnwokeka%2FKDE-Telepathy-presence-plasmoid.git&a=summary" target="_blank">http://quickgit.kde.org/?p=scratch%2Fnwokeka%2FKDE-Telepathy-presence-plasmoid.git&a=summary</a><br>
</div></div></div><br><div><br></div></div></blockquote><div><br></div><div>Been playing around with it, got some cool stuff - tested adding/removing accounts with it running, that all behaves. I'm quite pleased.</div>
<div> </div></div>Bug/TODO List:<div><br><div> - all the presences are in lower case (i.e "available" not "Available") as we are directly displaying id values. Not looked at your code, but I'd guess this means it's not translated either. It should probably go through a map of id -> translated pretty name. </div>
<div><br></div><div>(possibly this should be in the dataengine, it could have a global map of presence IDs -> presence icon + presence readable name.) </div><div><br></div><div> - Doesn't behave in a panel for me.</div>
<div><br></div><div> - I'm not sure you'd ever want to open the accounts KCM in such a hurry that we need a link. </div><div><br></div><div>- If you try to connect an account that then fails (i.e password wrong), the plasmoid still says "available".</div>
<div> We also don't get an error notification because the CL isn't running..but that's a much bigger issue for another day.</div><div><br></div><div>- You've made CMake check that the contact-list is installed - as we have a button to launch it. This isn't a "compile dependency" so shouldn't really be in the CMake file. It should either be - a dependency at a packaging level, or ideally a run time check and only show the button if it's available (not sure we can do that with plasma JS bindings).</div>
<div><br></div><div>- Would be really cool if we could see the protocol icon next to the name of the account somewhere. I think this requires a dataengine change. </div><div><br></div><div>- /All/ the avatars flash when any connection changes</div>
<div> </div><div>Having said that, I'm 100% happy for you to push to review (and then over the top of the old repo after review) as it currently is. Then we can slowly fix all the tiny issues together.</div><div><br></div>
<div>Dave</div><div><br></div></div>