After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 750287 - Make OSK touch friendly (on wayland)
Make OSK touch friendly (on wayland)
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: keyboard
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2015-06-02 15:20 UTC by Carlos Garnacho
Modified: 2015-06-04 11:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Revert "Revert "keyboard: Handle touch events"" (1.90 KB, patch)
2015-06-02 15:21 UTC, Carlos Garnacho
accepted-commit_now Details | Review

Description Carlos Garnacho 2015-06-02 15:20:23 UTC
We're currently only handling pointer events on OSK keys, which coincidentally works on X11 (as pointer events are emulated out of the XITouchEmulatingPointer touch sequence). This doesn't happen on wayland though, where touch is not expected to emulate pointer events.

I'm attaching a patch (actually an older one revisited by rui/me) to handle touch events on wayland. This is left aside in X11 as there's weird interaction with the passive touch grab that don't make it as easy to obey one or the other, so it will be saner to stick to pointer-only events there.
Comment 1 Carlos Garnacho 2015-06-02 15:21:10 UTC
Created attachment 304442 [details] [review]
Revert "Revert "keyboard: Handle touch events""

And make these only handled on wayland. There's a plethora of issues
around touch passive grab and touch/pointer doubly handling to use
these right away on X11, so we stick to single-touch/pointer there.

This reverts commit 032a688a72110c7f7b976a5b3bd03c3feab32646.
Comment 2 Rui Matos 2015-06-02 15:27:32 UTC
Review of attachment 304442 [details] [review]:

Right, on X11 we'll keep working via the emulated pointer events for touch while on native ("wayland") where we just get touch events we can at least use the OSK. Please push for 3.16 as well
Comment 3 Jasper St. Pierre (not reading bugmail) 2015-06-02 15:36:53 UTC
Review of attachment 304442 [details] [review]:

::: js/ui/keyboard.js
@@ +120,3 @@
+                let sequence = event.get_event_sequence();
+
+                if (!Meta.is_wayland_compositor())

A comment here would be nice.
Comment 4 Carlos Garnacho 2015-06-02 15:48:39 UTC
(In reply to Jasper St. Pierre from comment #3)
> Review of attachment 304442 [details] [review] [review]:
> 
> ::: js/ui/keyboard.js
> @@ +120,3 @@
> +                let sequence = event.get_event_sequence();
> +
> +                if (!Meta.is_wayland_compositor())
> 
> A comment here would be nice.

Sure, adding one and pushing.
Comment 5 Carlos Garnacho 2015-06-02 15:59:18 UTC
Attachment 304442 [details] pushed as a7b0910 - Revert "Revert "keyboard: Handle touch events""
Comment 6 Bastien Nocera 2015-06-04 11:22:54 UTC
Can we also land this in GNOME 3.16?

I can't login with the default GDM configuration (which uses Wayland), on a tablet. It worked in 3.14 when gdm used X11.