Review Request: Porting of presence applet to c++ backend

[Francesco Nwokeka]

> On Dec. 13, 2011, 1:19 p.m., Martin Klapetek wrote:
> > That's an interesting turn in design :) Is this still full widget (expanded on click) or is it only the icon controlled via context menu now? I agree with George that it shouldn't be in submenu though.

No popup plasmoid, just the icon with the presence. The context menu has been modified with g.k's comment.
Two more new features will be implemented as well (presence message on hover + left click-> contact list).


> On Dec. 13, 2011, 1:19 p.m., Martin Klapetek wrote:
> > Screenshot: prev2 - the presence plasmoid context menu
> > <http://git.reviewboard.kde.org>
> >
> >     Ooh, is that the Afro Samurai?

Oh yeah!


- Francesco


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103362/#review8912
-----------------------------------------------------------


On Dec. 12, 2011, 12:31 p.m., Francesco Nwokeka wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103362/
> -----------------------------------------------------------
> 
> (Updated Dec. 12, 2011, 12:31 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> This is the begining of the port from dataengine -> c++. I've decided to do this because the communication between QML and dataengine is quite crappy.
> With a c++ backend the plasmoid is waaaaaay MORE flexible and can access any information in less time and is more reliable.
> 
> Atm the c++ doesn't communicate with the QML part (which is missing. This will be added after I meet with Dario to talk about some design issues etc) atm but will be done as soon as some minor things are out of the way
> 
> http://quickgit.kde.org/?p=scratch%2Fnwokeka%2Fpresence-applet-porting.git&a=summary
> 
> 
> Diffs
> -----
> 
> 
> Diff: http://git.reviewboard.kde.org/r/103362/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> prev1 - desktop plasmoid
>   http://git.reviewboard.kde.org/r/103362/s/364/
> prev2 - the presence plasmoid context menu
>   http://git.reviewboard.kde.org/r/103362/s/365/
> prev3 - systray
>   http://git.reviewboard.kde.org/r/103362/s/366/
> 
> 
> Thanks,
> 
> Francesco Nwokeka
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20111213/6bd17b89/attachment.html>