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

input/cursor: correctly send tablet v2 up event when over non-v2 surface #5237

Merged

Conversation

Xyene
Copy link
Member

@Xyene Xyene commented Apr 19, 2020

If we started holding the tool tip down on a surface that accepts tablet
v2, we should notify that surface if it gets released over a surface
that doesn't support v2.

Since GTK supports tablet v2, this fixes the common case of starting a
drag over a GTK surface (e.g. scrollbar) and releasing it outside (e.g.
over the gaps between sway containers, or in a terminal).

It does not fix the other direction, e.g. dragging over a non-v2 surface
(with tablet events being emulated as mouse events) and releasing over
a v2 surface.

Refs #5230.

If we started holding the tool tip down on a surface that accepts tablet
v2, we should notify that surface if it gets released over a surface
that doesn't support v2.

Since GTK supports tablet v2, this fixes the common case of starting a
drag over a GTK surface (e.g. scrollbar) and releasing it outside (e.g.
over the gaps between sway containers, or in a terminal).

Refs swaywm#5230.
Copy link
Member

@emersion emersion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

It would be nice to have a more robust plan for pointer emulation in general.

@emersion emersion merged commit d88460f into swaywm:master Apr 22, 2020
@Xyene Xyene deleted the fix-tablet-tool-release-cursor-emulation branch October 18, 2020 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants