Review Request 108643: Fix screenlocker lockwindow geometry on screen resizes (take 2)

[Thomas Lübking]

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

Ship it!


Please revert the forceActiveWindow call to activateWindow (for mentioned reason) and ship this in as many commits as you feel required (at least the code parts that update the screen amount and force 0,0,0,255 instead of 0,0,0,0)

In a second pass (and for master) we can then clean up the code from legacy stuff and figure why you encounter a 32bit drawable here (or this is maybe an issue in the driver)

- Thomas Lübking


On Jan. 28, 2013, 2:38 p.m., Aaron J. Seigo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108643/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2013, 2:38 p.m.)
> 
> 
> Review request for kwin, Plasma, Martin Gräßlin, and Thomas Lübking.
> 
> 
> Description
> -------
> 
> Updated version of Thomas' patch from https://git.reviewboard.kde.org/r/108585/
> 
> Tested and works in the use case originally noted as broken, and as a bonus includes getting rid of usage of libkephal.
> 
> 
> This addresses bug 313835.
>     http://bugs.kde.org/show_bug.cgi?id=313835
> 
> 
> Diffs
> -----
> 
>   ksmserver/screenlocker/CMakeLists.txt bab32eda33526bbe5149bc903fbb6a28fb9d28d7 
>   ksmserver/screenlocker/greeter/CMakeLists.txt f89a98d66b774d00bbe8b881c66298ed7e339dac 
>   ksmserver/screenlocker/greeter/greeterapp.cpp b41cc74ce4f072610772366be3abdf1f4f25f24e 
>   ksmserver/screenlocker/lockwindow.h 5cbc76956a4b83d46da96a7ef768116ffdd505c3 
>   ksmserver/screenlocker/lockwindow.cpp 147853b5625277a35bf01247366c6597636c638c 
> 
> Diff: http://git.reviewboard.kde.org/r/108643/diff/
> 
> 
> Testing
> -------
> 
> start locker when internal laptop screen is on
> plug in external screen
> kscreen automagically switches it on and the laptop screen off
> locker displays properly
> 
> 
> Thanks,
> 
> Aaron J. Seigo
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130129/78d45390/attachment.html>