Bug 417604 - Scrolling direction inverted in some Plasma scrollviews
Summary: Scrolling direction inverted in some Plasma scrollviews
Status: RESOLVED UPSTREAM
Alias: None
Product: plasmashell
Classification: Plasma
Component: general (other bugs)
Version First Reported In: 5.18.0
Platform: Manjaro Linux
: HI major
Target Milestone: 1.0
Assignee: David Edmundson
URL: https://codereview.qt-project.org/c/q...
Keywords: regression, usability, wayland-only
: 417686 417694 421951 422366 423202 423375 423550 427966 428494 428919 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-02-13 23:08 UTC by Pietro F. Fontana
Modified: 2020-11-20 14:27 UTC (History)
22 users (show)

See Also:
Latest Commit: https://codereview.qt-project.org/c/qt/qtwayland/+/305647
Version Fixed/Implemented In: Qt 5.15.2
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Pietro F. Fontana 2020-02-13 23:08:34 UTC 
SUMMARY
Since few KDE Plasma versions I observed that setting the scrolling direction between most applications (Firefox, Dolphin, Okular, Konsole) and some Plasma specific applications (network applet, system settings) is inverted.
If I enable "natural scrolling" then this applies correctly to most applications but not to Plasma applets etc., if I disable the situation is the opposite.

STEPS TO REPRODUCE
1. Scroll in System Settings and in Dolphin (for example)
2. Enable/Disable "natural scrolling"
3. Scroll again

OBSERVED RESULT
The direction of scrolling is different in the two applications.

EXPECTED RESULT
The direction of scrolling is always the same in every application.

SOFTWARE/OS VERSIONS 
Operating System: Manjaro Linux 
KDE Plasma Version: 5.18.0
KDE Frameworks Version: 5.67.0
Qt Version: 5.14.1
Kernel Version: 5.4.19-1-MANJARO
OS Type: 64-bit

ADDITIONAL INFORMATION
Plasma on Wayland
Comment 1 postix 2020-02-15 10:42:55 UTC 
*** Bug 417686 has been marked as a duplicate of this bug. ***
Comment 2 postix 2020-02-15 10:47:08 UTC 
*** Bug 410156 has been marked as a duplicate of this bug. ***
Comment 3 Michał Dybczak 2020-02-15 17:53:51 UTC 
I can confirm this issue and point out that more relevant data can be found in this duplicate post: Bug 410156. So if you are any of the developers trying to debug and fix it, make sure you check this one.
Comment 4 Benjamin Smith 2020-02-15 18:51:11 UTC 
From my experience, this bug affects Plasma Wayland only on Qt 5.14.x. It started for me in Plasma 5.17 when Manjaro & Arch updated to Qt 5.14. It still persists in Plasma 5.18 with Qt 5.14.1. The bug is not present in Kubuntu 19.10 [with Plasma 5.17] or 20.04-nightly [with Plasma 5.18] because they build against Qt 5.12.x.
Comment 5 Patrick Silva 2020-02-16 04:47:39 UTC 
*** Bug 417694 has been marked as a duplicate of this bug. ***
Comment 6 Patrick Silva 2020-02-16 04:49:04 UTC 
Same thing on Wayand session of Neon unstable after update to Qt 5.14.1.

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.18.80
KDE Frameworks Version: 5.68.0
Qt Version: 5.14.1
Comment 7 David Edmundson 2020-04-06 21:57:27 UTC 
Debugged this a bit.

It seems Qt in QtWayland inverts the angle delta, but not the pixel delta.
This doesn't make much sense.
Comment 8 David Edmundson 2020-04-07 11:09:25 UTC 
Edit. ignore that. It is a bug compositor side.

axis and discrete axis should go the same way, yet WAYLAND_DEBUG shows they're not.

It also works correctly when running native, so it's an issue either in libinput or processing data from it.
Comment 9 David Edmundson 2020-04-07 16:51:28 UTC 
Edit again: I shouldn't doubt myself. I was right initially

https://codereview.qt-project.org/c/qt/qtwayland/+/296523

Hopefully will be in Qt5.15.0, we'll see
Comment 10 Nate Graham 2020-04-07 16:57:06 UTC 
Thanks David!
Comment 11 Patrick Silva 2020-05-23 19:32:44 UTC 
*** Bug 421951 has been marked as a duplicate of this bug. ***
Comment 12 Patrick Silva 2020-06-02 23:58:09 UTC 
*** Bug 422366 has been marked as a duplicate of this bug. ***
Comment 13 79253435110 2020-06-03 06:48:52 UTC 
> Hopefully will be in Qt5.15.0, we'll see

It looks like 5.15.0 has the same issue.
Comment 14 madcatx 2020-06-13 15:51:59 UTC 
Just FYI, I just patched QtWayland 5.15 with the upstream patch and the issue did not go away. Has anybody else experienced this?
Comment 15 Nate Graham 2020-06-13 16:22:04 UTC 
I am also still experiencing the issue with Qt 5.15. But I guess that makes sense since https://codereview.qt-project.org/c/qt/qtwayland/+/296523 hasn't actually been merged into Qt yet.
Comment 16 madcatx 2020-06-13 16:46:16 UTC 
(In reply to Nate Graham from comment #15)
> I am also still experiencing the issue with Qt 5.15. But I guess that makes
> sense since https://codereview.qt-project.org/c/qt/qtwayland/+/296523 hasn't
> actually been merged into Qt yet.

A patched QtWayland myself with the proposed patch and it did *not* fix the problem. Maybe there is another issue with Qt 5.15 that needs to be looked at?
Comment 17 Patrick Silva 2020-06-19 21:03:13 UTC 
*** Bug 423202 has been marked as a duplicate of this bug. ***
Comment 18 Patrick Silva 2020-06-23 16:37:29 UTC 
*** Bug 423375 has been marked as a duplicate of this bug. ***
Comment 19 Patrick Silva 2020-06-27 05:52:51 UTC 
*** Bug 423550 has been marked as a duplicate of this bug. ***
Comment 20 Reagan Bohan 2020-07-21 23:48:34 UTC 
Also patched with upstream patch, the issue was not fixed.
Comment 21 David Edmundson 2020-07-24 16:13:22 UTC 
>Also patched with upstream patch, the issue was not fixed.

patch updated for all axis sources
Comment 22 Reagan Bohan 2020-07-25 06:32:49 UTC 
(In reply to David Edmundson from comment #21)
> patch updated for all axis sources

Where was this updated? I still see abandoned on the 25th of june here: https://codereview.qt-project.org/c/qt/qtwayland/+/296523
Comment 23 Reagan Bohan 2020-07-27 05:18:48 UTC 
(In reply to David Edmundson from comment #21)
> patch updated for all axis sources

I have applied the new patch and the scrolling direction is correct. A few issues still persist: The scrolling jumps back to the top (Probably a QT bug) and the speed is still different (might need to be patched here).
Comment 24 Nate Graham 2020-07-27 14:39:24 UTC 
Here is the new Qt patch: https://codereview.qt-project.org/c/qt/qtwayland/+/305647
Comment 25 Nate Graham 2020-10-11 02:54:09 UTC 
That patch was merged, and I've got Qt 5.15.1 yet this is still happening to me on Wayland. The latest comment in there says this:


Failed to cherry pick to 5.15.
Reason: 409: Cannot create new patch set of change 296523 because it is abandoned


So I guess that means it only went to Qt 6? Can yo manually
Comment 26 Nate Graham 2020-10-11 02:54:22 UTC 
*can you manually cherry-pick it?
Comment 27 Patrick Silva 2020-10-19 17:12:58 UTC 
*** Bug 427966 has been marked as a duplicate of this bug. ***
Comment 28 Patrick Silva 2020-11-01 01:52:20 UTC 
*** Bug 428494 has been marked as a duplicate of this bug. ***
Comment 29 Nate Graham 2020-11-10 18:17:12 UTC 
*** Bug 428919 has been marked as a duplicate of this bug. ***
Comment 30 postix 2020-11-20 14:24:05 UTC 
Fixed in 5.15.2

https://code.qt.io/cgit/qt/qtwayland.git/tree/dist/changes-5.15.2/?h=v5.15.2
 - [QTBUG-85767] Touchpad scrolling is no longer inverted.