Review Request: [Patch 1/2] Fix Sliding Popups

[Martin Gräßlin]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103366/
-----------------------------------------------------------

Review request for kwin and Plasma.


Description
-------

When the offset was not explicitly specified, the actual distance of the window to the nearest screen edge should be used as offset. This was calculated incorrectly resulting in broken code in the kwin effect.

With this change a new "default" is introduced as -1 which is interpreted by kwin as kwin has to calculate the correct offset. Now it would be possible to do this in the library and pass the correct offset, but the library would have to create a QDesktopWidget just to calculate the screen geometry. KWin on the other hand knows the screen size, so having the default is IMHO better.

Adjustments to kwin come in next patch for kde-workspace


This addresses bug 287602.
    http://bugs.kde.org/show_bug.cgi?id=287602


Diffs
-----

  plasma/windoweffects.cpp ce41176 

Diff: http://git.reviewboard.kde.org/r/103366/diff/diff


Testing
-------


Thanks,

Martin Gräßlin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111209/f08deede/attachment-0001.html>