Review Request 125226: move EventForge from the desktop containment
Marco Martin
notmart at gmail.com
Tue Sep 15 17:30:50 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125226/
-----------------------------------------------------------
(Updated Sept. 15, 2015, 5:30 p.m.)
Review request for KDE Frameworks, Plasma and Eike Hein.
Changes
-------
RFC: this version impements a similar thing in MouseEventListener, that would be done automatically, without API.
there may be side effects, but it may make sense if a mouseeventlistener manages a pressandhold, it cancels all the events half managed by the children so far
Repository: kdeclarative
Description
-------
EventForge can cancel the half-managed events of child items
in order to implement the move of them with press and hold,
not having the parent and the children battling for the same
press-move-release event sequence.
API-wise I think it's fine, apart perhaps the EventForge class name
(perhaps MouseEventGrabber is less technically correct, but still more clear?)
Diffs (updated)
-----
src/qmlcontrols/kquickcontrolsaddons/mouseeventlistener.h a62efb7
src/qmlcontrols/kquickcontrolsaddons/mouseeventlistener.cpp d3321a3
Diff: https://git.reviewboard.kde.org/r/125226/diff/
Testing
-------
Thanks,
Marco Martin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150915/8b1f810a/attachment.html>
More information about the Plasma-devel
mailing list