[effects] Forward wheel events
ClosedPublic

Authored by davidedmundson on Nov 13 2019, 5:38 PM.

Details

Summary

Effects::checkInputWindowEvent(QWheelEvent *e) existed but it was not
hooked up to anything and would never be called.

This patch adds the relevant handling in the wayland filter, X filter so
the existing method gets called.

EffectQuickView is updated to handle wheel events.

Test Plan

Used a ListView via an EffectQuickView in an effect.
I can now scroll with the mouse.

Diff Detail

Repository
R108 KWin
Branch
master
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 18786
Build 18804: arc lint + arc unit
davidedmundson created this revision.Nov 13 2019, 5:38 PM
Restricted Application added a project: KWin. · View Herald TranscriptNov 13 2019, 5:38 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
davidedmundson requested review of this revision.Nov 13 2019, 5:38 PM
zzag added a subscriber: zzag.Nov 13 2019, 6:24 PM
zzag added inline comments.
plugins/platforms/x11/standalone/effects_mouse_interception_x11_filter.cpp
42

You forgot horizontal scroll events, i.e. button 6 and button 7.

zzag added inline comments.Nov 13 2019, 6:26 PM
plugins/platforms/x11/standalone/effects_mouse_interception_x11_filter.cpp
42

button 6 - left
button 7 - right

Add horizontal mouse wheel.

We're using magic numbers as 6 and 7 aren't defined in
xcb headers. (though BUTTON_5 wasn't much better than 5
as a semantic description anyway)

Left and right values look backwards but it's copied from
Qt... I can't test that (which is why I hoped originally
to subtly just avoid dealing this situation!)

zzag accepted this revision.Nov 14 2019, 12:30 PM
This revision is now accepted and ready to land.Nov 14 2019, 12:30 PM
This revision was automatically updated to reflect the committed changes.