From 2d28e4f3f2bd0fac70ecff83407bfd636e6e727d Mon Sep 17 00:00:00 2001
From: MartyIX <203266+MartyIX@users.noreply.github.com>
Date: Tue, 9 Jul 2024 22:40:19 +0200
Subject: [PATCH] Subscribe pointer events only when needed

---
 .../GesturePlatformManager.Windows.cs         | 39 ++++++++++++-------
 1 file changed, 24 insertions(+), 15 deletions(-)

diff --git a/src/Controls/src/Core/Platform/GestureManager/GesturePlatformManager.Windows.cs b/src/Controls/src/Core/Platform/GestureManager/GesturePlatformManager.Windows.cs
index ba6d8a98f69c..34bc58064405 100644
--- a/src/Controls/src/Core/Platform/GestureManager/GesturePlatformManager.Windows.cs
+++ b/src/Controls/src/Core/Platform/GestureManager/GesturePlatformManager.Windows.cs
@@ -730,15 +730,7 @@ void PinchComplete(bool success)
 
 		void UpdateDragAndDropGestureRecognizers()
 		{
-			if (_container is null)
-			{
-				return;
-			}
-
-			var view = Element as View;
-			IList<IGestureRecognizer>? gestures = view?.GestureRecognizers;
-
-			if (gestures is null)
+			if (_container is null || Element is not View view || view.GestureRecognizers is not IList<IGestureRecognizer> gestures)
 			{
 				return;
 			}
@@ -812,16 +804,18 @@ void UpdatingGestureRecognizers()
 				}
 			}
 
-			_subscriptionFlags |= SubscriptionFlags.ContainerPgrPointerEventsSubscribed;
-			_container.PointerEntered += OnPgrPointerEntered;
-			_container.PointerExited += OnPgrPointerExited;
-			_container.PointerMoved += OnPgrPointerMoved;
-			_container.PointerPressed += OnPgrPointerPressed;
-			_container.PointerReleased += OnPgrPointerReleased;
+			bool hasPointerGesture = ElementGestureRecognizers.HasAnyGesturesFor<PointerGestureRecognizer>();
+
+			if (hasPointerGesture)
+			{
+				_subscriptionFlags |= SubscriptionFlags.ContainerPgrPointerEventsSubscribed;
+				SubscribePointerEvents(_container);
+			}
 
 			bool hasSwipeGesture = gestures.HasAnyGesturesFor<SwipeGestureRecognizer>();
 			bool hasPinchGesture = gestures.HasAnyGesturesFor<PinchGestureRecognizer>();
 			bool hasPanGesture = gestures.HasAnyGesturesFor<PanGestureRecognizer>();
+
 			if (!hasSwipeGesture && !hasPinchGesture && !hasPanGesture)
 			{
 				return;
@@ -840,6 +834,12 @@ void UpdatingGestureRecognizers()
 				return;
 			}
 
+			if (!hasPointerGesture)
+			{
+				_subscriptionFlags |= SubscriptionFlags.ContainerPgrPointerEventsSubscribed;
+				SubscribePointerEvents(_container);
+			}
+
 			_subscriptionFlags |= SubscriptionFlags.ContainerManipulationAndPointerEventsSubscribed;
 			_container.ManipulationMode = ManipulationModes.Scale | ManipulationModes.TranslateX | ManipulationModes.TranslateY;
 			_container.ManipulationDelta += OnManipulationDelta;
@@ -848,6 +848,15 @@ void UpdatingGestureRecognizers()
 			_container.PointerCanceled += OnPointerCanceled;
 		}
 
+		void SubscribePointerEvents(FrameworkElement container)
+		{
+			container.PointerEntered += OnPgrPointerEntered;
+			container.PointerExited += OnPgrPointerExited;
+			container.PointerMoved += OnPgrPointerMoved;
+			container.PointerPressed += OnPgrPointerPressed;
+			container.PointerReleased += OnPgrPointerReleased;
+		}
+
 		void HandleTapped(object sender, TappedRoutedEventArgs tappedRoutedEventArgs)
 		{
 			tappedRoutedEventArgs.Handled = true;