Review Request: Add some feature to presence icon

Commit Hook null at kde.org
Fri Jan 27 13:32:44 UTC 2012


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


This review has been submitted with commit 922d07d6e4599320a6791913c3ecdc6a57460b2e by Daniele E. Domenichelli to branch master.

- Commit Hook


On Jan. 27, 2012, 12:50 p.m., Daniele Elmo Domenichelli wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103802/
> -----------------------------------------------------------
> 
> (Updated Jan. 27, 2012, 12:50 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> The method "KIcon Presence::icon() const" becomes "KIcon Presence::icon(bool useImIcons = false) const" that allows you to use im-user-* icons instead of the user-* (i.e. the "pawn" icons instead of the "circles")
> Adds a method "KIcon Presence::icon(QStringList overlays, bool useImIcons = false) const" that allows to construct the icon with overlays.
> Finally adds a method "QString Presence::icon(bool useImIcons = false) const" to get the name of the icon for the current presence (useful if you want to create an icon using the presence as overlay)
> 
> 
> WARNING: This patch breaks the ABI (Not API though) and you will have to recompile the presence-applet or it will make plasma-desktop crash at next login if you have one in the taskbar (possibly also if you have it in the desktop)
> 
> 
> Diffs
> -----
> 
>   KTp/presence.h 07f3576e75b75cf76c038776e55f7c02a3cda395 
>   KTp/presence.cpp d291a812e1521b295a5c5d147b152140f5a17411 
> 
> Diff: http://git.reviewboard.kde.org/r/103802/diff/diff
> 
> 
> Testing
> -------
> 
> Works
> 
> 
> Thanks,
> 
> Daniele Elmo Domenichelli
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120127/9b18f6af/attachment.html>


More information about the KDE-Telepathy mailing list