----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113124/#review43917 -----------------------------------------------------------
I'd rather not commit something like this that compares pointers when Nicolás has already pointed out that its very posible for addresses to be reused, maybe someone can do some work with the event accepted flag? If it's the same object we can just set it to accepted the first time and then ignore the others? - Albert Astals Cid On Oct. 6, 2013, 7:36 p.m., Nicolás Alvarez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113124/ > ----------------------------------------------------------- > > (Updated Oct. 6, 2013, 7:36 p.m.) > > > Review request for Plasma. > > > Bugs: 316546 > http://bugs.kde.org/show_bug.cgi?id=316546 > > > Repository: kde-runtime > > > Description > ------- > > MouseEventListener: Don't process event from children multiple times. > > If an event passed through multiple visually-overlapping children of > MouseEventListener, the event filter was processing the event every time. > For example, this caused problems for the KMix icon in the tray, which > would trigger volume changes three times for every mouse wheel event. > > BUG:316546 > > > Diffs > ----- > > plasma/declarativeimports/qtextracomponents/mouseeventlistener.cpp > 36530e075dc52348f32032c32457a090b904c3b9 > > Diff: http://git.reviewboard.kde.org/r/113124/diff/ > > > Testing > ------- > > The KMix problem is gone, each mouse wheel step now changes the volume by the > appropriate amount. > > > Thanks, > > Nicolás Alvarez > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
