Review Request 126331: Left click on item in task manager is ignored, arbitrary

[David Edmundson]

On Dec. 13, 2015, 1:26 p.m., Anthony Fieroni wrote:
> > If we just revert that last commit in here (which is basically what this change is doing) we go back to reintroducing that bug. 
> > 
> > I'll merge this before 5.5.1 if we don't have anything else, but I'd rather we had both things fixed properly not just bouncing round between having 2 different bugs.

oh, you're the same author as: https://git.reviewboard.kde.org/r/126175/

Does this patch fix the issue you're trying to fix there?


- David


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


On Dec. 13, 2015, 12:30 p.m., Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126331/
> -----------------------------------------------------------
> 
> (Updated Dec. 13, 2015, 12:30 p.m.)
> 
> 
> Review request for Plasma and David Edmundson.
> 
> 
> Bugs: 354651
>     https://bugs.kde.org/show_bug.cgi?id=354651
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Last patch has a reggression due to QtQuick issue (only on Xcb): QEvent::Leave is triggered after QEvent::MouseButtonPress Qt::LeftButton
> 
> 
> Diffs
> -----
> 
>   shell/panelview.cpp 063343d 
> 
> Diff: https://git.reviewboard.kde.org/r/126331/diff/
> 
> 
> Testing
> -------
> 
> This QtQuick (Xcb) issue (on Wayland works fine) can be found and on Comic Strip ('show arrows on mouse over' in settings) every first click on arrow is ignored. I'm not familiar with Qt, if this bug is fixed or Qt devs must be noticed for this behavior. During patch testing i found other bug, when drag an item from task manager and drop in the  desktop, on desktop is created a shortcut, but this works only when widgets are unlocked, if they are locked drop cause a crash, a backtrace present.
> 
> 
> File Attachments
> ----------------
> 
> plasmashell-20151213-132332.kcrash.txt
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/12/13/3720870f-3827-41c3-b522-1a2ac5a5611a__plasmashell-20151213-132332.kcrash.txt
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

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