Skip to content

Fix pan and pinch gesture completion on Windows - #31123

Merged
jfversluis merged 3 commits into
dotnet:inflight/currentfrom
jeremy-visionaid:pinch-gesture-completion-fix
Aug 21, 2025
Merged

jfversluis merged 3 commits into
dotnet:inflight/currentfrom
jeremy-visionaid:pinch-gesture-completion-fix

Conversation

@jeremy-visionaid

@jeremy-visionaid jeremy-visionaid commented Aug 12, 2025

Copy link
Copy Markdown
Contributor

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Description of Change

Fixes various issues with completion events not being received and pinch gestures not operating correctly on Windows.

Issues Fixed

Fixes #31033

  • Pinches running with only one touch point when interrupted by a popup/alert
  • Pan completion events not being received when more than one touch point is specified
  • Pinches becoming inoperable before both touch points are removed

@jeremy-visionaid
jeremy-visionaid requested a review from a team as a code owner August 12, 2025 07:22
@dotnet-policy-service dotnet-policy-service Bot added the community ✨ Community Contribution label Aug 12, 2025
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Hey there @@jeremy-visionaid! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@jeremy-visionaid
jeremy-visionaid marked this pull request as draft August 12, 2025 07:28
@jeremy-visionaid

Copy link
Copy Markdown
Contributor Author

@MartyIX Oh, I realize now that the panning is supposed to continue running even when the cursor exits the control. I'll have another go...

@jeremy-visionaid
jeremy-visionaid force-pushed the pinch-gesture-completion-fix branch from ce96b38 to dfef008 Compare August 12, 2025 07:47
@jeremy-visionaid
jeremy-visionaid force-pushed the pinch-gesture-completion-fix branch from dfef008 to 0aa3b46 Compare August 12, 2025 08:23
@jeremy-visionaid

Copy link
Copy Markdown
Contributor Author

OK, this should now fix up a few corner cases:

  1. Pinches running with only one touch point when interrupted by a popup/alert
  2. Pan completion events not being received when more than one touch point is specified
  3. Pinches becoming inoperable before both touch points are removed

@jeremy-visionaid
jeremy-visionaid marked this pull request as ready for review August 12, 2025 08:27
@jfversluis

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 3 pipeline(s).

@jfversluis
jfversluis changed the base branch from main to inflight/current August 21, 2025 18:42
@jfversluis jfversluis added this to the .NET 9 SR12 milestone Aug 21, 2025
@jfversluis
jfversluis merged commit c4564b1 into dotnet:inflight/current Aug 21, 2025
127 of 129 checks passed
PureWeen pushed a commit that referenced this pull request Sep 5, 2025
Remove unnecessary guards in GesturePlatformManager

Fix Windows pan and pinch gesture completion

Fixes: #31033
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

community ✨ Community Contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PinchGestureRecognizer does not complete correctly when interrupted by an alert on Windows

2 participants