> 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 > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
