> 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