Review Request: Add some feature to presence icon

[Martin Klapetek]

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



KTp/presence.cpp
<http://git.reviewboard.kde.org/r/103802/#comment8354>

    I don't think the 'switch' is really needed here, it can be much clearer with just
    
    if (!iconString().isEmpty) {
        return KIcon(iconString()...)
    }
    
    return KIcon();



KTp/presence.cpp
<http://git.reviewboard.kde.org/r/103802/#comment8355>

    Same as above.


- Martin Klapetek


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/7dcf1369/attachment-0001.html>