----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129263/#review100297 -----------------------------------------------------------
Ship it! Ship It! - Marco Martin On Oct. 26, 2016, 10:56 a.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129263/ > ----------------------------------------------------------- > > (Updated Oct. 26, 2016, 10:56 a.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > And only enable it in the contextMenuAboutToShow connection. > > We currently enable/disable the menu in this slot, but we default to it on. > This meant other users of actions() that can't correctly emit > contextMenuAboutToShow gain an extra entry which might not be relevant. > > > Diffs > ----- > > src/plasma/private/applet_p.cpp bfb6d2502a2fc804497e571e7000e20a40a20d5c > > Diff: https://git.reviewboard.kde.org/r/129263/diff/ > > > Testing > ------- > > Appears on kickoff > Doesn't appear on the system tray > > (as before) > > No longer appears in the toolbox (new fix) > > > Thanks, > > David Edmundson > >