----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115157/ -----------------------------------------------------------
Review request for KDE Frameworks, Martin Gräßlin and Michael Palimaka. Repository: knotifications Description ------- This is an alternative approach to https://git.reviewboard.kde.org/r/115147/ where we avoid doing the cross platform support ourselves, and instead depend on KWindowSystem. Downsides: it looks like the implementations for KWindowInfo::geometry() and KWindowInfo::frameGeometry are the wrong way round on windows, and they are not implemented at all on Mac. However, the code we had here was never tested either, so... Make better use of KWindowSystem in KPassivePopup This avoids having code that is compiled on non-X11 platforms but not on X11 (the old non-X11 code did not compile). Diffs ----- CMakeLists.txt 022cbcb7a12aa5ad9843019ffd73f1b3e117fb9b src/kpassivepopup.cpp b41cb8dc8b3372346bd12c0413bf6bfa9a6fd00d tests/CMakeLists.txt c7481362008e3cae10d0afcfbcaea5fe953ce62d tests/kpassivepopuptest.cpp f457aed7e57904bcc00462a947bc5eaae7208792 Diff: https://git.reviewboard.kde.org/r/115157/diff/ Testing ------- Compiles and tests work when X11 is found. Compiles and tests do as well as expected (ie: popups are placed next to the window instead of the taskbar entry) when X11 is not found (specifically by commenting out find_package(X11)). Thanks, Alex Merry
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel