KEMBAR78
Fix pan and pinch gesture completion on Windows by jeremy-visionaid · Pull Request #31123 · dotnet/maui · GitHub
Skip to content

Conversation

jeremy-visionaid
Copy link
Contributor

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

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
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
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
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
Member

/azp run

@azure-pipelines
Copy link

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