D8864: RFC: [MouseEventListener] Allow accepting mouse event
Kai Uwe Broulik
noreply at phabricator.kde.org
Fri Nov 17 14:07:28 UTC 2017
broulik created this revision.
broulik added reviewers: Plasma, hein.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY
This will keep the event from propagating. Accepting a press event will also not result in clicked or pressAndHold being handled.
In Qt 5.10 event propagation changed resulting in FolderView opening both the item context menu and containment context menu. Imho this actually makes sense since we never accepted the mouse event there.
TEST PLAN
I put an `mouse.accepted` in FolderView after opening the context menu and no longer get double menus showing up.
However, I'm not sure how delicate this change is. In theory it shouldn't change existing behavior and fixing/changing FV might be the better solution?
REPOSITORY
R296 KDeclarative
REVISION DETAIL
https://phabricator.kde.org/D8864
AFFECTED FILES
src/qmlcontrols/kquickcontrolsaddons/mouseeventlistener.cpp
src/qmlcontrols/kquickcontrolsaddons/mouseeventlistener.h
To: broulik, #plasma, hein
Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171117/80f0350e/attachment.html>
More information about the Plasma-devel
mailing list