> On Дек. 1, 2015, 11 преди обяд, David Edmundson wrote:
> > Looks good. Thanks for that.
> > 
> > Do you have commmit access?
> 
> Anthony Fieroni wrote:
>     No, i hope will see it for 5.5 :)
> 
> David Edmundson wrote:
>     I've just moved a bit of code so it's now: https://paste.kde.org/pmebpx2tm
>     
>     you ok with that?
> 
> Anthony Fieroni wrote:
>     About me, this will NOT work. Why you do this? Will work if remove line 
> 31 :) Mouse move is triggered, but cursor is outside the panel - this code 
> will not make nothing.
> 
> David Edmundson wrote:
>     We only get mouse events outside the panel in two conditions:
>      - leave events (which we still have)
>      - drags (which we dont' want to generated new leave events on)

Ok, seems legit :)


- Anthony


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


On Ноев. 30, 2015, 10:26 след обяд, Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126175/
> -----------------------------------------------------------
> 
> (Updated Ноев. 30, 2015, 10:26 след обяд)
> 
> 
> Review request for Plasma, David Edmundson and Marco Martin.
> 
> 
> Bugs: 354651
>     https://bugs.kde.org/show_bug.cgi?id=354651
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> You can see videos in bug repot
> 
> 
> Diffs
> -----
> 
>   shell/panelview.cpp 3407501 
> 
> Diff: https://git.reviewboard.kde.org/r/126175/diff/
> 
> 
> Testing
> -------
> 
> I test patch and all works as expected.
> http://doc.qt.io/qt-5/qevent.html
> QEvent::Leave 11      Mouse leaves widget's boundaries.
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to