xpra icon
Bug tracker and wiki

Attachments (3)

pointer.html (3.0 KB) - added by Antoine Martin 11 months ago.
javascript test page
pointer.2.html (5.4 KB) - added by Antoine Martin 11 months ago.
updated test page
pointer.3.html (1.5 KB) - added by Antoine Martin 11 months ago.
minimal example

Download all attachments as: .zip

Change History (15)

comment:1 Changed 11 months ago by Antoine Martin

Status: newassigned

On regular browsers, we see pairs of events:

new: pointermove PointerEvent {isTrusted: true, pointerId: 1, width: 1, height: 1, pressure: 0, …}
new: mousemove MouseEvent {isTrusted: true, screenX: 430, screenY: 238, clientX: 430, clientY: 109, …}

With ms edge when using two finger scrolling, we see just the pointermove.

Changed 11 months ago by Antoine Martin

Attachment: pointer.html added

javascript test page

Changed 11 months ago by Antoine Martin

Attachment: pointer.2.html added

updated test page

Changed 11 months ago by Antoine Martin

Attachment: pointer.3.html added

minimal example

comment:2 Changed 11 months ago by Antoine Martin

Basic workaround merged in r21198.

Still TODO: make it an option so we can choose between "zoom" and "scroll" modes for the touchpad since we can't have both... and don't enable those workarounds unless we really need them (ie: other browsers don't).
That's because the touch-action attribute doesn't allow us to intercept scroll events without also intercepting zoom events.

The proper way to deal with this would be to forward all touch events to the server (#1615), without emulating a wheel event, so that the server could take the appropriate action based on the touch gesture.

Related: there are issues with uinput support (see ticket:1611#comment:29) so we don't support "smooth scrolling" via uinput.

Last edited 11 months ago by Antoine Martin (previous) (diff)

comment:3 Changed 11 months ago by Antoine Martin

r21200 allows pointer events to bubble up, which allows most browsers to zoom anywhere, just not msedge..

comment:4 Changed 11 months ago by Antoine Martin

Owner: changed from Antoine Martin to J. Max Mena
Status: assignednew

r21208 adds a rather ugly top bar menu entry so that msedge users can choose between scrolling and zooming with touch events.

comment:5 Changed 11 months ago by J. Max Mena

Essentially, I do not have access to a Windows 10 device with MS Edge installed to test with at this time. I'm working to update my Windows 10 tablet, but it looks like it's caught in a Microsoft catch 22:

Need disk space to update, so updates have hung.

This machine only has a 16GB Windows partition, and it's only got 3GB free and the only things I have installed on it are Google Chrome and FTL (A game totaling ~700MB).

I'll ponder this one for a bit. If this ticket is urgent, please pass it to Alex.

comment:6 Changed 11 months ago by Antoine Martin

Essentially, I do not have access to a Windows 10 device with MS Edge installed to test with at this time.

It's not just msedge you need but a "precision touchpad" device.

comment:7 Changed 11 months ago by J. Max Mena

Okay, in that case I don't think I own a single device that qualifies.

comment:8 Changed 10 months ago by Antoine Martin

Fix for Firefox in r21510.

comment:9 Changed 9 months ago by Antoine Martin

I've kept this change in my local tree - but I'm not sure what for..

--- html5/css/client.css	(revision 21584)
+++ html5/css/client.css	(working copy)
@@ -7,7 +7,7 @@
 }
 
 div.window canvas {
-	touch-action: none;
+	touch-action: auto;
 }
 
 body.desktop {

comment:10 Changed 9 months ago by Antoine Martin

Owner: changed from J. Max Mena to Jonathan Anthony

comment:11 Changed 3 months ago by Smo

Owner: changed from Jonathan Anthony to Smo

comment:12 Changed 3 weeks ago by Antoine Martin

Resolution: worksforme
Status: newclosed
Note: See TracTickets for help on using tickets.