For whatever reason Qt scrolled the wrong way on internal windows. As a
new QWheelEvent is constructed anyway, just multiply by -1 to get the
direction fixed.
BUG: 371999
FIXED-IN: 5.8.4
Details
Details
- Reviewers
broulik - Group Reviewers
Plasma on Wayland KWin - Commits
- R108:da0ba7632483: Invert the scroll direction for wheel events on internal windows
Tried debug console in nested setup, scrolled correctly now.
Diff Detail
Diff Detail
- Repository
- R108 KWin
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
How does that affect horizontal scrolling? Also, does that already take into account scroll direction "natural"/normal (or was that maybe the case for it being wrong in the first place?)?
Comment Actions
The same: as we multiple by -1 it's also corrected now. Though I admit to not have tested it (yet)
Also, does that already take into account scroll direction "natural"/normal (or was that maybe the case for it being wrong in the first place?)?
The cause in the first place is that Qt expects the delta in another dimension. I just verified by looking at Qt code and there it's also multiplied by a negative value: https://code.qt.io/cgit/qt/qtwayland.git/tree/src/client/qwaylandinputdevice.cpp#n557
Comment Actions
Verified that horizontal scrolling also scrolls correctly (dragged one TreeWidget colum super wide to have it scroll)