----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118019/#review57421 -----------------------------------------------------------
src/kdbusservice.cpp <https://git.reviewboard.kde.org/r/118019/#comment39953> obviously this should also only be executed on platform xcb - Martin Gräßlin On May 6, 2014, 4:35 p.m., Sebastian Kügler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118019/ > ----------------------------------------------------------- > > (Updated May 6, 2014, 4:35 p.m.) > > > Review request for KDE Frameworks, David Faure and Martin Gräßlin. > > > Bugs: 332532 > https://bugs.kde.org/show_bug.cgi?id=332532 > > > Repository: kdbusaddons > > > Description > ------- > > Update X11 app timestamp before activating window: This patch updates the > window's timestamp before sending the signal to the app to raise the window. > This makes raising the window following requestActivated work. It needs a > handler in the app, though, to do the actual call to > QMainWindow->activateWindow(). > > This part at least *could* go into KDBusAddons, perhaps conditionally > (HAVE_X11?). Otherwise, we'd have to do this also in every single app that > requires it. > > Feedback? > > > Diffs > ----- > > CMakeLists.txt fe5feaf536894aeac821a35ad91817da8a834bc6 > src/CMakeLists.txt 7c0c48506e350eebc742719f76dcb4f8b0a74816 > src/kdbusservice.cpp 4239e8582e1f578523e17e76985eae5159c35c0c > > Diff: https://git.reviewboard.kde.org/r/118019/diff/ > > > Testing > ------- > > Implemented handler in systemsettings, verified that the window is raised on > second start. > > > Thanks, > > Sebastian Kügler > >
_______________________________________________ Kde-frameworks-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
