----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107545/#review22876 -----------------------------------------------------------
Ship it! Thanks, just spotted one minor coding style issue I missed in the first review, so please fix it before committing src/presenceapplet.cpp <http://git.reviewboard.kde.org/r/107545/#comment17449> Move "{" to new line - Dan Vrátil On Dec. 1, 2012, 4:35 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, 4:35 p.m.) > > > Review request for Telepathy. > > > 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/ > > > Thanks, > > Andromeda Galaxy > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
