Review Request: Correctly display popups when using multiple screens

Matthias Fuchs mat69 at gmx.net
Thu Jul 28 16:33:38 UTC 2011


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

Review request for Plasma and Aaron J. Seigo.


Summary
-------

If there are multiple screens with different resolutions or which
are not alligned the same way then it could happen that popups at
the edge were drawn (partially) offscreen.
This patch fixes that issue.

I have the feeling that the code in this method is really ugly, I wonder if there are plans to refactor it or to clean it up?

NOTE: I don't know if this patch might cause problems if there are animations that are supposed to start offscreen etc. So please you with more insight look at the patch to see if there could be some negative side effects.


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


Diffs
-----

  plasma/corona.cpp 4afef7b 

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


Testing
-------


Thanks,

Matthias

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110728/64c004c5/attachment.htm 


More information about the Plasma-devel mailing list