----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106353/#review18614 -----------------------------------------------------------
Screenshot: <http://git.reviewboard.kde.org//r/106353/#scomment87> Icon context-menu.cpp <http://git.reviewboard.kde.org/r/106353/#comment14716> This isn't per contact.. you're not passing any contact information to the configwidget... - David Edmundson On Sept. 6, 2012, 11:01 a.m., Rohan Garg wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106353/ > ----------------------------------------------------------- > > (Updated Sept. 6, 2012, 11:01 a.m.) > > > Review request for Telepathy. > > > Description > ------- > > Add context menu entry to modify contact notifications > > The position of the configure entry is debatable, should it be at the bottom > with "Show info" ? I'd like to hear your opinions :) > > > Diffs > ----- > > CMakeLists.txt ffadc0bea1aa2cfd1f5222b06d2289775d0921ba > context-menu.h 5a30c6d326c7401d7f226fdca1cb90b65d650403 > context-menu.cpp 10f3d90cc7de3c7ce5072d3a6dc449f9ecb8e721 > > Diff: http://git.reviewboard.kde.org/r/106353/diff/ > > > Testing > ------- > > Right click a contact > click "Configure Notifications" > Brings up > notification configuration dialog > > > Screenshots > ----------- > > > http://git.reviewboard.kde.org/r/106353/s/716/ > > > Thanks, > > Rohan Garg > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
