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.

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

Wheel-scrolling over remote assistance breaks input #2572

Closed
Saviq opened this issue Aug 23, 2022 · 5 comments
Closed

Wheel-scrolling over remote assistance breaks input #2572

Saviq opened this issue Aug 23, 2022 · 5 comments
Assignees

Comments

@Saviq
Copy link
Collaborator

Saviq commented Aug 23, 2022

With Frame from edge, or edge/mir-pr2566, and edge -VNC:

WAYLAND_DISPLAY=wayland-99 ubuntu-frame --add-wayland-extensions \
    zwlr_virtual_pointer_manager_v1:zwp_relative_pointer_manager_v1:zwp_virtual_keyboard_manager_v1 &
WAYLAND_DISPLAY=wayland-99 wpe-webkit-mir-kiosk.cog

In a separate shell (because of #2571)

WAYLAND_DISPLAY=wayland-99 ubuntu-frame-vnc

Connect a VNC client and scroll your mouse wheel "through" it.

The client complains:

interface 'wl_pointer' has no event 9

And input stops working.

@AlanGriffiths AlanGriffiths self-assigned this Aug 23, 2022
@AlanGriffiths
Copy link
Contributor

OK, so "event 9" is axis_value120 - i.e. Mir is sending wl_resource_post_event(resource, Opcode::axis_value120, ...) when the client doesn't expect it.

But all the Mir code I see gates this on wl_resource_get_version(resource) >= 8 - which (modulo hard coded numbers) should be correct.

@AlanGriffiths
Copy link
Contributor

This looks like a problem with wpe-webkit-mir-kiosk: it is negotiating wl_seat version 8 - which includes support for axis_value120, and then when it receives axis_value120 it complains. I'd expect the same behaviour without VNC, but prior to #2566 we were not sending the correct axis_value120 events on modern libinput.

@AlanGriffiths
Copy link
Contributor

Confirmed that this also affects HW input (with Mir dev, which now includes #2566).

@AlanGriffiths
Copy link
Contributor

With GNOME, is only offered at wl_seat at 5. But webkit doesn't display anyway

@AlanGriffiths
Copy link
Contributor

Reported against the wpe-webkit snap: https://gitlab.com/glancr/wpe-webkit-snap/-/issues/38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants