Review Request: Add plasma-themeability to the menu on the presence applet

[Martin Klapetek]

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


Usability folks - we have two sorts of presence menus, one is in the plasmoid, which is usually put in the systray and one in contact list. We recently added plasma themed presence icons for the plasmoid itself. This patch now adds those same plasma icons into the presence menu of the plasmoid, making this one looking different from the contact list one. Also, all other plasmoids use oxygen icons in their menus, so I'm inclined to keep the oxygen icons.

What's your opinion?

- Martin Klapetek


On Dec. 1, 2012, 5:39 p.m., Andromeda Galaxy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107545/
> -----------------------------------------------------------
> 
> (Updated Dec. 1, 2012, 5:39 p.m.)
> 
> 
> Review request for Telepathy and KDE Usability.
> 
> 
> Description
> -------
> 
> The Presence applet will show a Plasma-themed icon on the desktop now. However, the menu items for statuses are still all Oxygen, so the user experience is inconsistent -- if the user clicks on an icon for status, they won't get that icon on the toolbar. The attached diff, if applied to the master, will make it so that those menu items are also Plasma-themed.
> 
> 
> Diffs
> -----
> 
>   src/presenceapplet.h 78ccfbd 
>   src/presenceapplet.cpp 291cde4 
> 
> Diff: http://git.reviewboard.kde.org/r/107545/diff/
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> The plasma-themed applet
>   http://git.reviewboard.kde.org/r/107545/s/863/
> The presence dropdown on the contact list
>   http://git.reviewboard.kde.org/r/107545/s/864/
> 
> 
> Thanks,
> 
> Andromeda Galaxy
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20121201/905fb24c/attachment.html>