Review Request 115157: Make better use of KWindowSystem in KPassivePopup

Alex Merry kde at randomguy3.me.uk
Mon Jan 20 17:58:40 UTC 2014


-----------------------------------------------------------
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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140120/67dc7515/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list