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

Add passthrough touch support for windows native multi-touch #629

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ZigZagT
Copy link

@ZigZagT ZigZagT commented May 21, 2024

Adding support for sending touch events to Sunshine hosts directly.

This has been asked in multiple issues (incomplete list)

Summary of changes

  1. A new "passthrough touch events" option is added in settings when "Touchscreen" mode is on :
image

Once turned on, touch events will be routed to Sunshine hosts via the LiSendTouchEvent API, and appears as natively multi touch events on Windows.

  1. When passthrough mode is enabled, the pop-up keyboard would require 5 fingers tap to fire up. This avoids conflicting with windows native gestures. (According to the touch-gestures-for-windows, windows has native touch gestures use up to 4 fingers, so 5 fingers tap provides a good guarantee). Windows also offers a built-in on-screen touch keyboard when multi-touch events are received, making the pop-up keyboard less crucial than before.

  2. A performance tweak (turning off UIScrollView delaysContentTouches). While I was on this I noticed the "pin & pan & scroll" feature in absolute touch mode brings in unwanted delay due to the default settings of ScrollView, which might have caused the following issues regarding delays:

@ZigZagT ZigZagT force-pushed the master branch 5 times, most recently from 2e73fb1 to 0c81f2b Compare May 21, 2024 09:57
@stefanilijev97
Copy link

This is really cool and long awaited feature 😀 when we can expect this to make it to the official releases?

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

Successfully merging this pull request may close these issues.

3 participants