Review Request 125522: Use NETRootInfo to initiate wm move operation

Hugo Pereira Da Costa hugo.pereira at free.fr
Wed Oct 7 14:14:22 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125522/#review86450
-----------------------------------------------------------


Should also be checked when compiling against kde4 (cmake -DUSE_KDE4=1)
As far as I can tell, right now it does not compile there. 
Also, concerning the added release, and other hacks, this was needed because in the past at least, when dragging the window, you would immediatly after miss one click as well as hover highlight.
I'll double check if this is still the case (against kf5).

- Hugo Pereira Da Costa


On Oct. 4, 2015, 11:15 p.m., David Rosca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125522/
> -----------------------------------------------------------
> 
> (Updated Oct. 4, 2015, 11:15 p.m.)
> 
> 
> Review request for Plasma and Hugo Pereira Da Costa.
> 
> 
> Repository: breeze
> 
> 
> Description
> -------
> 
> Also remove hack that messes with cursor position, which results in window sometimes rendered in wrong position for a moment after starting window move.
> 
> I'm not sure why there was added a button release before ungrabbing pointer in startDrag(), but it works fine without it.
> 
> 
> Diffs
> -----
> 
>   kstyle/breezewindowmanager.h 7989052 
>   kstyle/breezewindowmanager.cpp a9ad7ee 
> 
> Diff: https://git.reviewboard.kde.org/r/125522/diff/
> 
> 
> Testing
> -------
> 
> Moving windows still works.
> I cannot reproduce the issue described in comment of the mentioned hack.
> 
> 
> Thanks,
> 
> David Rosca
> 
>

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


More information about the Plasma-devel mailing list