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

[Thomas Pfeiffer]

> On Dec. 1, 2012, 5:44 p.m., Martin Klapetek wrote:
> > 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?

First of all: Why do the Plasma-themed icons have to look so different from the Oxygen ones? Who says that Plasma-themed icons have to be speech bubbles? Wouldn't monochromatic versions of the Oxygen icons work as well? The only problem I see is that online and invisible are currently only distinguishable by color (maybe we can ask nuno to come up with a different icon for invisible).
If they were had the same form, it would be far less confusing for users to see different icon sets as they should be able to easily recognize them. 


- Thomas


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


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