[Windows] Fix pointer released event not being handled on ImageButton

src/Controls/samples/Controls.Sample.UITests/Issues/Issue21706.xaml

@@ -0,0 +1,40 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<ContentPage xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
+    xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
+    x:Class="Maui.Controls.Sample.Issues.Issue21706"
+    xmlns:ns="clr-namespace:Maui.Controls.Sample.Issues">
+    <VerticalStackLayout>
+        <ImageButton 
+            Source="groceries.png"
+            WidthRequest="200"
+            HeightRequest="200"
+            AutomationId="WaitForElement"
+            x:Name="PointerImageButton">
+            <VisualStateManager.VisualStateGroups>
+                <VisualStateGroup 
+                    x:Name="CommonStates">
+                    <VisualState 
+                        x:Name="Normal">
+                        <VisualState.Setters>
+                            <Setter 
+                                Property="BackgroundColor" Value="Blue" />
+                        </VisualState.Setters>
+                    </VisualState>
+                    <VisualState 
+                        x:Name="PointerOver">
+                        <VisualState.Setters>
+                            <Setter 
+                                Property="BackgroundColor" Value="Red" />
+                        </VisualState.Setters>
+                    </VisualState>
+                </VisualStateGroup>
+            </VisualStateManager.VisualStateGroups>
+        </ImageButton>
+
+        <Button
+            AutomationId="OtherButton"
+            WidthRequest="200"
+            HeightRequest="200"
+            Text="Other"/>
+    </VerticalStackLayout>
+</ContentPage>

src/Controls/samples/Controls.Sample.UITests/Issues/Issue21706.xaml.cs

@@ -0,0 +1,18 @@
+using System;
+using System.Collections.Generic;
+using System.ComponentModel;
+using Microsoft.Maui.Controls;
+using Microsoft.Maui.Controls.Xaml;
+
+namespace Maui.Controls.Sample.Issues
+{
+	[XamlCompilation(XamlCompilationOptions.Compile)]
+	[Issue(IssueTracker.Github, 21706, "ImageButton stuck in PointerOver state", PlatformAffected.UWP)]
+	public partial class Issue21706 : ContentPage
+	{
+		public Issue21706()
+		{
+			InitializeComponent();
+		}
+	}
+}

src/Controls/samples/Controls.Sample/Pages/Layouts/FlexLayoutPage.xaml

@@ -39,6 +39,6 @@
                 FontSize="Large"
                 BackgroundColor="Pink"
                 HorizontalTextAlignment="Center" />
-        </FlexLayout>
+            </FlexLayout>
     </views:BasePage.Content>
 </views:BasePage>

src/Controls/tests/UITests/Tests/Issues/Issue21706.cs

@@ -0,0 +1,26 @@
+using NUnit.Framework;
+using UITest.Appium;
+using UITest.Core;
+
+namespace Microsoft.Maui.AppiumTests.Issues
+{
+	public class Issue21706 : _IssuesUITest
+	{
+		public override string Issue => "ImageButton stuck in PointerOver state";
+
+		public Issue21706(TestDevice device) : base(device)
+		{
+		}
+
+		[Test]
+		public async Task ImageButtonStuckAfterRightClick()
+		{
+			this.IgnoreIfPlatforms(new TestDevice[] { TestDevice.Android, TestDevice.Mac, TestDevice.iOS });
+			App.WaitForElement("WaitForElement");
+			App.RightClick("WaitForElement");
+			App.Click("OtherButton");
+			await Task.Delay(500);
+			VerifyScreenshot();
+		}
+	}
+}

src/Core/src/Handlers/ImageButton/ImageButtonHandler.Windows.cs

@@ -11,6 +11,7 @@ public partial class ImageButtonHandler : ViewHandler<IImageButton, Button>
 		Image? _image;
 
 		PointerEventHandler? _pointerPressedHandler;
+		PointerEventHandler? _pointerReleasedHandler;
 
 		protected override Button CreatePlatformView()
 		{
@@ -34,6 +35,7 @@ protected override Button CreatePlatformView()
 		protected override void ConnectHandler(Button platformView)
 		{
 			_pointerPressedHandler = new PointerEventHandler(OnPointerPressed);
+			_pointerReleasedHandler = new PointerEventHandler(OnPointerReleased);
 
 			if (_image != null)
 			{
@@ -43,6 +45,7 @@ protected override void ConnectHandler(Button platformView)
 
 			platformView.Click += OnClick;
 			platformView.AddHandler(UIElement.PointerPressedEvent, _pointerPressedHandler, true);
+			platformView.AddHandler(UIElement.PointerReleasedEvent, _pointerReleasedHandler, true);
 
 			base.ConnectHandler(platformView);
 		}
@@ -57,8 +60,10 @@ protected override void DisconnectHandler(Button platformView)
 
 			platformView.Click -= OnClick;
 			platformView.RemoveHandler(UIElement.PointerPressedEvent, _pointerPressedHandler);
+			platformView.RemoveHandler(UIElement.PointerReleasedEvent, _pointerReleasedHandler);
 
 			_pointerPressedHandler = null;
+			_pointerReleasedHandler = null;
 
 			base.DisconnectHandler(platformView);
 
@@ -101,6 +106,11 @@ void OnPointerPressed(object sender, PointerRoutedEventArgs e)
 			VirtualView?.Pressed();
 		}
 
+		void OnPointerReleased(object sender, PointerRoutedEventArgs e)
+		{
+			VirtualView?.Released();
+		}
+
 		void OnImageOpened(object sender, RoutedEventArgs routedEventArgs)
 		{
 			VirtualView?.UpdateIsLoading(false);

src/TestUtils/src/UITest.Appium/HelperExtensions.cs

@@ -23,6 +23,11 @@ public static void Click(this IApp app, string element)
 			app.FindElement(element).Click();
 		}
 
+		public static void RightClick(this IApp app, string element)
+		{
+			app.FindElement(element).RightClick();
+		}
+
 		public static string? GetText(this IUIElement element)
 		{
 			var response = element.Command.Execute("getText", new Dictionary<string, object>()
@@ -119,6 +124,15 @@ public static void Click(this IUIElement element)
 			});
 		}
 
+		public static void RightClick(this IUIElement element)
+		{
+			element.Command.Execute("click", new Dictionary<string, object>()
+			{
+				{ "element", element },
+				{ "button", "right" }
+			});
+		}
+
 		public static void SendKeys(this IUIElement element, string text)
 		{
 			element.Command.Execute("sendKeys", new Dictionary<string, object>()