xpra icon
Bug tracker and wiki

Attachments (3)

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

Download all attachments as: .zip

Change History (13)

comment:1 Changed 3 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 3 months ago by Antoine Martin

Attachment: pointer.html added

javascript test page

Changed 2 months ago by Antoine Martin

Attachment: pointer.2.html added

updated test page

Changed 2 months ago by Antoine Martin

Attachment: pointer.3.html added

minimal example

comment:2 Changed 2 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 2 months ago by Antoine Martin (previous) (diff)

comment:3 Changed 2 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 2 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 2 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 2 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 2 months ago by J. Max Mena

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

comment:8 Changed 3 weeks ago by Antoine Martin

Fix for Firefox in r21510.

comment:9 Changed 2 weeks 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 2 weeks ago by Antoine Martin

Owner: changed from J. Max Mena to Jonathan Anthony
Note: See TracTickets for help on using tickets.