Fix secondary tap on Android (#10619)

Co-authored-by: Rachel Kang <rachelkang@microsoft.com>

src/Controls/samples/Controls.Sample/Pages/Core/TapGestureGalleryPage.cs

@@ -12,6 +12,7 @@ public class TapGestureGalleryPage : BasePage
 		Label relativeToToggleButtonPosition = new Label();
 		Label relativeToContainerPosition = new Label();
 		Button toggleButton;
+		int clickCount = 0;
 
 		public TapGestureGalleryPage()
 		{
@@ -184,6 +185,8 @@ void OnTapped(object sender, System.EventArgs e)
 		void HandleTapCommand(Color backgroundColor)
 		{
 			changeColorBoxView.BackgroundColor = backgroundColor;
+			changeColorBoxView.Text = $"Tap Gesture Gallery: {clickCount}";
+			clickCount++;
 		}
 	}
 }

src/Controls/src/Core/Platform/Android/Extensions/MotionEventExtensions.cs

@@ -8,6 +8,15 @@ namespace Microsoft.Maui.Controls.Platform;
 
 internal static class MotionEventExtensions
 {
+  public static bool IsSecondary(this MotionEvent me)
+  {
+    var buttonState = me?.ButtonState ?? MotionEventButtonState.Primary;
+
+    return
+      buttonState == MotionEventButtonState.Secondary ||
+      buttonState == MotionEventButtonState.StylusSecondary;
+  }
+
 	internal static Point? CalculatePosition(this MotionEvent? e, IElement? sourceElement, IElement? relativeElement)
 	{
 		var context = sourceElement?.Handler?.MauiContext?.Context;

src/Controls/src/Core/Platform/Android/InnerGestureListener.cs

@@ -161,7 +161,12 @@ bool GestureDetector.IOnDoubleTapListener.OnSingleTapConfirmed(MotionEvent e)
 			if (_disposed)
 				return false;
 
-			if (!HasDoubleTapHandler())
+			// The secondary button state only surfaces inside `OnSingleTapConfirmed`
+			// Inside 'OnSingleTap' the e.ButtonState doesn't indicate a secondary click
+			// So, if the gesture recognizer here has primary/secondary we want to ignore
+			// the _tapDelegate call that accounts for secondary clicks
+			if (!HasDoubleTapHandler() &&
+				(!e.IsSecondary() || HasSingleTapHandlerWithPrimaryAndSecondary()))
 			{
 				// We're not worried about double-tap, so OnSingleTapUp has already run the delegate
 				// there's nothing for us to do here
@@ -236,6 +241,21 @@ internal void EndScrolling()
 			_isScrolling = false;
 		}
 
+		bool HasSingleTapHandlerWithPrimaryAndSecondary()
+		{
+			if (_tapGestureRecognizers == null)
+				return false;
+
+			var check = ButtonsMask.Primary | ButtonsMask.Secondary;
+			foreach (var gesture in _tapGestureRecognizers(1))
+			{
+				if ((gesture.Buttons & check) == check)
+					return true;
+			}
+
+			return false;
+		}
+
 		bool HasDoubleTapHandler()
 		{
 			if (_tapGestureRecognizers == null)
@@ -247,6 +267,7 @@ bool HasSingleTapHandler()
 		{
 			if (_tapGestureRecognizers == null)
 				return false;
+
 			return _tapGestureRecognizers(1).Any();
 		}
 	}