Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Mouse scrolling events are sent twice on GNOME #2494

Open
Friz64 opened this issue Feb 23, 2024 · 2 comments · May be fixed by #2499
Open

Mouse scrolling events are sent twice on GNOME #2494

Friz64 opened this issue Feb 23, 2024 · 2 comments · May be fixed by #2499
Assignees
@elmindreda
Labels
bug Bug reports and bugfix pull requests external Issues GLFW cannot reasonably solve input Keyboard, joystick or mouse verified Reproduced or otherwise verified bugs Wayland
Projects
Queue
Milestone
3.4.1

Comments

@Friz64
Copy link

Friz64 commented Feb 23, 2024

When scrolling with a mouse (not a laptop touchpad), the scroll events are sent twice. Reproduced on two different machines with two different mice using the events test. Couldn't reproduce on KDE Plasma.

Scrolling Up -> Down -> Up -> Down:

00000002 to 1 at 1.867: Scroll: 0.000 1.000
00000003 to 1 at 1.867: Scroll: 0.000 1.000
00000004 to 1 at 2.262: Scroll: 0.000 -1.000
00000005 to 1 at 2.262: Scroll: 0.000 -1.000
00000006 to 1 at 2.672: Scroll: 0.000 1.000
00000007 to 1 at 2.672: Scroll: 0.000 1.000
00000008 to 1 at 3.103: Scroll: 0.000 -1.000
00000009 to 1 at 3.103: Scroll: 0.000 -1.000

Here's WAYLAND_DEBUG=1 output if it helps: scrolling-wayland-debug.txt
@elmindreda elmindreda added bug Bug reports and bugfix pull requests Wayland input Keyboard, joystick or mouse labels Feb 23, 2024
@elmindreda elmindreda self-assigned this Feb 23, 2024
@elmindreda elmindreda changed the title Mouse scrolling events are sent twice on GNOME Wayland Mouse scrolling events are sent twice on GNOME Feb 23, 2024
@Friz64 Friz64 linked a pull request Feb 25, 2024 that will close this issue
Draft
@BoyOrigin
Copy link

I also noticed that way before 2024, but I "fixed" the problem myself.
https://github.com/BoyOrigin/glfw-wayland/blob/main/patches/0002-Fix-duplicate-pointer-scroll-events.patch

@elmindreda
Copy link
Member

This seems likely to be caused by a bug in GNOME's implementation of version 4 of wl_seat, especially considering that #2499 resolves the issue without any of the additional logic a move to version 5 would require. This is great news because I was going to implement support for at least version 8 soon anyway, which should then also fix this bug.

@elmindreda elmindreda added verified Reproduced or otherwise verified bugs external Issues GLFW cannot reasonably solve labels Apr 12, 2024
@elmindreda elmindreda added this to Todo in Queue via automation Apr 12, 2024
@elmindreda elmindreda moved this from Todo to Quick, Isolated and Approved in Queue Apr 12, 2024
@elmindreda elmindreda added this to the 3.4.1 milestone Apr 12, 2024
@arturamiryan arturamiryan mentioned this issue Apr 14, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@elmindreda elmindreda

Labels
bug Bug reports and bugfix pull requests external Issues GLFW cannot reasonably solve input Keyboard, joystick or mouse verified Reproduced or otherwise verified bugs Wayland
Projects
Queue
Quick, Isolated and Approved
Milestone
3.4.1
Development

Successfully merging a pull request may close this issue.

Fix duplicate scroll events Friz64/glfw
3 participants
@elmindreda @Friz64 @BoyOrigin