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

[David Edmundson]

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



shell/panelview.cpp (line 762)
<https://git.reviewboard.kde.org/r/126331/#comment61182>

    this is basically the same as the line as you just added above? (but using the right position?)
    
    we certainly don't need both.



shell/panelview.cpp (line 930)
<https://git.reviewboard.kde.org/r/126331/#comment61181>

    why is this change here?


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.

- David Edmundson


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/e92d861e/attachment.html>