-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118019/
-----------------------------------------------------------
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
