----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118200/#review58144 -----------------------------------------------------------
This review has been submitted with commit c5c34851dc410aa14663cf3d8d0ea14fd061d31b by Martin Gräßlin to branch master. - Commit Hook On May 19, 2014, 10:56 a.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118200/ > ----------------------------------------------------------- > > (Updated May 19, 2014, 10:56 a.m.) > > > Review request for KDE Frameworks, Plasma and Marco Martin. > > > Repository: knotifications > > > Description > ------- > > [StatusNotifier] Don't set custom window flags on ContextMenu > > A QMenu has proper default flags with BypassWindowManagerHint. > Thus all the setting of hints and force activate is not needed. > Also it takes care of grabbing keyboard and pointer in this case > resulting in the popup being closed when clicked outside. > > This fixes interaction problems with Klipper. > > > Diffs > ----- > > src/kstatusnotifieritemdbus_p.cpp 0073afacfc3c0a5b6164e0e777471e1d5ea3cb43 > > Diff: https://git.reviewboard.kde.org/r/118200/diff/ > > > Testing > ------- > > Klipper properly grabs the pointer and clicking outside Klipper's context > menu, ends it. > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
