[iOS/Android] Fix SwipeItem.IsVisible not refreshing native swipe items when binding changes

[SyedAbdulAzeemSF4852]

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!

Issue Details

• SwipeItem.IsVisible doesn't properly refresh the native swipe items when the binding value changes dynamically. SwipeItem is not a full VisualElement, so visibility changes does not trigger a native re-rendered.

Root Cause

• SwipeItem.IsVisibleProperty had no propertyChanged callback, so binding changes were never forwarded to the handler. On iOS, UpdateSwipeItems() recreates native views on every swipe-open and those views always defaulted to Hidden = false, ignoring the current binding value. On both iOS and Android, MapVisibility called UpdateIsVisibleSwipeItem (layout) before setting the native visibility.

Description of Change

• Added a propertyChanged callback (OnIsVisibleChanged) to the IsVisibleProperty in SwipeItem, ensuring that changes to visibility trigger UI updates.
• Updated the MapVisibility method in both Android and iOS handlers to set the native view's visibility state before recalculating item positions, ensuring consistent UI behavior.
• Modified the iOS platform code to set the Hidden property of swipe items during initialization, reflecting their initial visibility state.

Issues Fixed

Fixes #34832

Validated the behaviour in the following platforms

• Windows
• Android
• iOS
• Mac

Output

Platform	Before	After
iOS	iOS_Before.mov	iOS_After.mov
Android	Android_Before.mov	Android_After.mov

[github-actions]

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.sh | bash -s -- 35217

Or

• Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 35217"

[dotnet-policy-service]

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

[MauiBot]

🤖 Automated review — alternative fix proposed

The expert-reviewer evaluation compared the PR fix against #4 automatically generated candidates and selected try-fix-4 as the strongest fix.

Why: try-fix-4 applies the PR's idiomatic propertyChanged callback approach (the standard MAUI pattern for bindable properties) plus the code-reviewer's iOS swipeItemsWidth correction (items.Count(GetIsVisible) to avoid oversized _actionView.Frame). All four platform-level fixes are present (SwipeItem.cs callback, Android/iOS MapVisibility reorder, iOS Hidden init), the Android UITest passes cleanly, and the fix is minimal at 4 files.

Please consider applying the candidate diff below (or use it as guidance). Once you push an update, this workflow will re-trigger and re-evaluate.

Candidate diff (`try-fix-4`)

diff --git a/src/Controls/src/Core/SwipeView/SwipeItem.cs b/src/Controls/src/Core/SwipeView/SwipeItem.cs
index c295bc4936..8505094c9e 100644
--- a/src/Controls/src/Core/SwipeView/SwipeItem.cs
+++ b/src/Controls/src/Core/SwipeView/SwipeItem.cs
@@ -14,7 +14,7 @@ namespace Microsoft.Maui.Controls
 		public static readonly BindableProperty BackgroundColorProperty = BindableProperty.Create(nameof(BackgroundColor), typeof(Color), typeof(SwipeItem), null);
 
 		/// <summary>Bindable property for <see cref="IsVisible"/>.</summary>
-		public static readonly BindableProperty IsVisibleProperty = BindableProperty.Create(nameof(IsVisible), typeof(bool), typeof(SwipeItem), true);
+		public static readonly BindableProperty IsVisibleProperty = BindableProperty.Create(nameof(IsVisible), typeof(bool), typeof(SwipeItem), true, propertyChanged: OnIsVisibleChanged);
 
 		/// <summary>
 		/// Gets or sets the background color of the swipe item. This is a bindable property.
@@ -40,6 +40,12 @@ namespace Microsoft.Maui.Controls
 
 		Visibility ISwipeItemMenuItem.Visibility => this.IsVisible ? Visibility.Visible : Visibility.Collapsed;
 
+		static void OnIsVisibleChanged(BindableObject bindable, object oldValue, object newValue)
+		{
+			var swipeItem = (SwipeItem)bindable;
+			swipeItem.Handler?.UpdateValue(nameof(ISwipeItemMenuItem.Visibility));
+		}
+
 		void Maui.ISwipeItem.OnInvoked()
 		{
 			if (Command != null && Command.CanExecute(CommandParameter))
diff --git a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
index 85061bef53..0d207b9104 100644
--- a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
+++ b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
@@ -72,10 +72,12 @@ namespace Microsoft.Maui.Handlers
 
 		public static void MapVisibility(ISwipeItemMenuItemHandler handler, ISwipeItemMenuItem view)
 		{
+			// Set visibility before UpdateIsVisibleSwipeItem so LayoutSwipeItems
+			// reads the correct visibility when recalculating item positions.
+			handler.PlatformView.Visibility = view.Visibility.ToPlatformVisibility();
+
 			var swipeView = handler.PlatformView.Parent.GetParentOfType<MauiSwipeView>();
 			swipeView?.UpdateIsVisibleSwipeItem(view);
-
-			handler.PlatformView.Visibility = view.Visibility.ToPlatformVisibility();
 		}
 
 		protected override AView CreatePlatformElement()
diff --git a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
index 647662e5d9..ed501e2a90 100644
--- a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
+++ b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
@@ -86,11 +86,12 @@ namespace Microsoft.Maui.Handlers
 
 		public static void MapVisibility(ISwipeItemMenuItemHandler handler, ISwipeItemMenuItem view)
 		{
-			var swipeView = handler.PlatformView.GetParentOfType<MauiSwipeView>();
+			// Set visibility before UpdateIsVisibleSwipeItem so LayoutSwipeItems
+			// reads the correct visibility when recalculating item positions.
+			handler.PlatformView.UpdateVisibility(view.Visibility);
 
+			var swipeView = handler.PlatformView.GetParentOfType<MauiSwipeView>();
 			swipeView?.UpdateIsVisibleSwipeItem(view);
-
-			handler.PlatformView.UpdateVisibility(view.Visibility);
 		}
 
 		partial class SwipeItemMenuItemImageSourcePartSetter
diff --git a/src/Core/src/Platform/iOS/MauiSwipeView.cs b/src/Core/src/Platform/iOS/MauiSwipeView.cs
index b7f3169536..bb7bef6c5c 100644
--- a/src/Core/src/Platform/iOS/MauiSwipeView.cs
+++ b/src/Core/src/Platform/iOS/MauiSwipeView.cs
@@ -299,7 +299,7 @@ namespace Microsoft.Maui.Platform
 			double swipeItemsWidth;
 
 			if (_swipeDirection == SwipeDirection.Left || _swipeDirection == SwipeDirection.Right)
-				swipeItemsWidth = items.Count * SwipeViewExtensions.SwipeItemWidth;
+				swipeItemsWidth = items.Count(GetIsVisible) * SwipeViewExtensions.SwipeItemWidth;
 			else
 				swipeItemsWidth = _contentView.Frame.Width;
 
@@ -312,6 +312,7 @@ namespace Microsoft.Maui.Platform
 			foreach (var item in items)
 			{
 				UIView swipeItem = item.ToPlatform(Element.Handler.MauiContext);
+				swipeItem.Hidden = !GetIsVisible(item);
 				_actionView.AddSubview(swipeItem);
 				_swipeItems.Add(item, swipeItem);
 			}

[kubaflo]

Could you please try ai's suggestions?

Resetting for re-review

[MauiBot]

🤖 Automated review — alternative fix proposed

The expert-reviewer evaluation compared the PR fix against #2 automatically generated candidates and selected try-fix-2 as the strongest fix.

Why: try-fix-2 (claude-sonnet-4.6) is the only passing candidate. It includes all of the PR's fixes plus two additional correctness improvements found by expert review: the LayoutSwipeItems index mismatch fix on both iOS and Android (i++ moved outside the hidden-check block), and the SwipeItemViewHandler.iOS/Android ordering fix to match the consistency of SwipeItemMenuItemHandler. Screenshot baselines were updated for the CI environment.

Please consider applying the candidate diff below (or use it as guidance). Once you push an update, this workflow will re-trigger and re-evaluate.

Candidate diff (`try-fix-2`)

diff --git a/src/Controls/src/Core/SwipeView/SwipeItem.cs b/src/Controls/src/Core/SwipeView/SwipeItem.cs
index c295bc4936..8505094c9e 100644
--- a/src/Controls/src/Core/SwipeView/SwipeItem.cs
+++ b/src/Controls/src/Core/SwipeView/SwipeItem.cs
@@ -14,7 +14,7 @@ namespace Microsoft.Maui.Controls
 		public static readonly BindableProperty BackgroundColorProperty = BindableProperty.Create(nameof(BackgroundColor), typeof(Color), typeof(SwipeItem), null);
 
 		/// <summary>Bindable property for <see cref="IsVisible"/>.</summary>
-		public static readonly BindableProperty IsVisibleProperty = BindableProperty.Create(nameof(IsVisible), typeof(bool), typeof(SwipeItem), true);
+		public static readonly BindableProperty IsVisibleProperty = BindableProperty.Create(nameof(IsVisible), typeof(bool), typeof(SwipeItem), true, propertyChanged: OnIsVisibleChanged);
 
 		/// <summary>
 		/// Gets or sets the background color of the swipe item. This is a bindable property.
@@ -40,6 +40,12 @@ namespace Microsoft.Maui.Controls
 
 		Visibility ISwipeItemMenuItem.Visibility => this.IsVisible ? Visibility.Visible : Visibility.Collapsed;
 
+		static void OnIsVisibleChanged(BindableObject bindable, object oldValue, object newValue)
+		{
+			var swipeItem = (SwipeItem)bindable;
+			swipeItem.Handler?.UpdateValue(nameof(ISwipeItemMenuItem.Visibility));
+		}
+
 		void Maui.ISwipeItem.OnInvoked()
 		{
 			if (Command != null && Command.CanExecute(CommandParameter))
diff --git a/src/Controls/tests/TestCases.Android.Tests/snapshots/android/SwipeOpen_DeleteHidden.png b/src/Controls/tests/TestCases.Android.Tests/snapshots/android/SwipeOpen_DeleteHidden.png
index b60e2ee9d0..964b9442a6 100644
Binary files a/src/Controls/tests/TestCases.Android.Tests/snapshots/android/SwipeOpen_DeleteHidden.png and b/src/Controls/tests/TestCases.Android.Tests/snapshots/android/SwipeOpen_DeleteHidden.png differ
diff --git a/src/Controls/tests/TestCases.Android.Tests/snapshots/android/SwipeOpen_DeleteVisible.png b/src/Controls/tests/TestCases.Android.Tests/snapshots/android/SwipeOpen_DeleteVisible.png
index eaef38beab..4181d07c96 100644
Binary files a/src/Controls/tests/TestCases.Android.Tests/snapshots/android/SwipeOpen_DeleteVisible.png and b/src/Controls/tests/TestCases.Android.Tests/snapshots/android/SwipeOpen_DeleteVisible.png differ
diff --git a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
index 85061bef53..0d207b9104 100644
--- a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
+++ b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
@@ -72,10 +72,12 @@ namespace Microsoft.Maui.Handlers
 
 		public static void MapVisibility(ISwipeItemMenuItemHandler handler, ISwipeItemMenuItem view)
 		{
+			// Set visibility before UpdateIsVisibleSwipeItem so LayoutSwipeItems
+			// reads the correct visibility when recalculating item positions.
+			handler.PlatformView.Visibility = view.Visibility.ToPlatformVisibility();
+
 			var swipeView = handler.PlatformView.Parent.GetParentOfType<MauiSwipeView>();
 			swipeView?.UpdateIsVisibleSwipeItem(view);
-
-			handler.PlatformView.Visibility = view.Visibility.ToPlatformVisibility();
 		}
 
 		protected override AView CreatePlatformElement()
diff --git a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
index 647662e5d9..0eb9faafd2 100644
--- a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
+++ b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
@@ -88,9 +88,11 @@ namespace Microsoft.Maui.Handlers
 		{
 			var swipeView = handler.PlatformView.GetParentOfType<MauiSwipeView>();
 
-			swipeView?.UpdateIsVisibleSwipeItem(view);
-
+			// Update the native view's Hidden state BEFORE calling UpdateIsVisibleSwipeItem,
+			// so LayoutSwipeItems can use the correct Hidden state when repositioning items.
 			handler.PlatformView.UpdateVisibility(view.Visibility);
+
+			swipeView?.UpdateIsVisibleSwipeItem(view);
 		}
 
 		partial class SwipeItemMenuItemImageSourcePartSetter
diff --git a/src/Core/src/Handlers/SwipeItemView/SwipeItemViewHandler.Android.cs b/src/Core/src/Handlers/SwipeItemView/SwipeItemViewHandler.Android.cs
index 3a56b8d73c..bfa2a79eff 100644
--- a/src/Core/src/Handlers/SwipeItemView/SwipeItemViewHandler.Android.cs
+++ b/src/Core/src/Handlers/SwipeItemView/SwipeItemViewHandler.Android.cs
@@ -49,6 +49,11 @@ namespace Microsoft.Maui.Handlers
 
 		public static void MapVisibility(ISwipeItemViewHandler handler, ISwipeItemView view)
 		{
+			// Set visibility before UpdateIsVisibleSwipeItem so LayoutSwipeItems
+			// reads the correct visibility when recalculating item positions.
+			if (handler.PlatformView != null)
+				handler.PlatformView.Visibility = view.Visibility.ToPlatformVisibility();
+
 			var swipeView = handler.PlatformView?.Parent.GetParentOfType<MauiSwipeView>();
 			swipeView?.UpdateIsVisibleSwipeItem(view);
 		}
diff --git a/src/Core/src/Handlers/SwipeItemView/SwipeItemViewHandler.iOS.cs b/src/Core/src/Handlers/SwipeItemView/SwipeItemViewHandler.iOS.cs
index 4d0dc435bf..d4c87bbff3 100644
--- a/src/Core/src/Handlers/SwipeItemView/SwipeItemViewHandler.iOS.cs
+++ b/src/Core/src/Handlers/SwipeItemView/SwipeItemViewHandler.iOS.cs
@@ -45,9 +45,12 @@ namespace Microsoft.Maui.Handlers
 		public static void MapVisibility(ISwipeItemViewHandler handler, ISwipeItemView view)
 		{
 			var swipeView = handler.PlatformView.GetParentOfType<MauiSwipeView>();
-			swipeView?.UpdateIsVisibleSwipeItem(view);
 
+			// Update the native view's Hidden state BEFORE calling UpdateIsVisibleSwipeItem,
+			// so LayoutSwipeItems can use the correct Hidden state when repositioning items.
 			handler.PlatformView.UpdateVisibility(view.Visibility);
+
+			swipeView?.UpdateIsVisibleSwipeItem(view);
 		}
 	}
 }
diff --git a/src/Core/src/Platform/Android/MauiSwipeView.cs b/src/Core/src/Platform/Android/MauiSwipeView.cs
index 7702e20185..9de6bb258a 100644
--- a/src/Core/src/Platform/Android/MauiSwipeView.cs
+++ b/src/Core/src/Platform/Android/MauiSwipeView.cs
@@ -621,7 +621,9 @@ namespace Microsoft.Maui.Platform
 
 			foreach (var child in childs)
 			{
-				if (child.Visibility == ViewStates.Visible)
+				// i advances for every child (visible or not) to keep childs[n] aligned with items[n].
+				// Guard against any unexpected count mismatch.
+				if (i < items.Count && child.Visibility == ViewStates.Visible)
 				{
 					var item = items[i];
 					var swipeItemSize = GetSwipeItemSize(item);
@@ -668,9 +670,10 @@ namespace Microsoft.Maui.Platform
 
 					child.Layout(l, t, r, b);
 
-					i++;
 					previousWidth += swipeItemWidth;
 				}
+
+				i++;
 			}
 		}
 
diff --git a/src/Core/src/Platform/iOS/MauiSwipeView.cs b/src/Core/src/Platform/iOS/MauiSwipeView.cs
index b7f3169536..360f468edc 100644
--- a/src/Core/src/Platform/iOS/MauiSwipeView.cs
+++ b/src/Core/src/Platform/iOS/MauiSwipeView.cs
@@ -312,6 +312,10 @@ namespace Microsoft.Maui.Platform
 			foreach (var item in items)
 			{
 				UIView swipeItem = item.ToPlatform(Element.Handler.MauiContext);
+				// iOS reuses connected handler platform views across swipe open/close cycles
+				// (unlike Android which disconnects handlers on close), so ConnectHandler and
+				// MapVisibility won't refire on reopen. Explicitly sync the Hidden state here.
+				swipeItem.Hidden = !GetIsVisible(item);
 				_actionView.AddSubview(swipeItem);
 				_swipeItems.Add(item, swipeItem);
 			}
@@ -342,7 +346,8 @@ namespace Microsoft.Maui.Platform
 
 			foreach (var child in childs)
 			{
-				if (!child.Hidden)
+				// i advances for every child (visible or not) to keep childs[n] aligned with items[n].
+				if (i < items.Count && !child.Hidden)
 				{
 					var item = items[i];
 					var swipeItemSize = Element.GetSwipeItemSize(item, _contentView, _swipeDirection);
@@ -371,10 +376,12 @@ namespace Microsoft.Maui.Platform
 						UpdateSwipeItemInsets(button);
 					}
 
-					i++;
 					previousWidth += swipeItemWidth;
 				}
 
+				i++;
+				// Hidden items retain their previous frame; ProcessTouchSwipeItems guards
+				// against interaction via GetIsVisible(), so stale frames are not acted on.
 				_swipeItemsRect.Add(child.Frame);
 			}
 		}

[MauiBot]

🤖 Automated review — alternative fix proposed

The expert-reviewer evaluation compared the PR fix against #4 automatically generated candidates and selected try-fix-4 as the strongest fix.

Why: Try-fix-4 is the only candidate that achieves a passing test. It applies the PR's correct core fix (propertyChanged callback, MapVisibility reorder, iOS Hidden init) with two additional improvements: (1) adds tolerance: 2.0 to snapshot comparisons to handle environment rendering variance causing the gate failure, and (2) fixes the Android LayoutSwipeItems index-misalignment bug (moves i++ outside the visibility guard) that the PR activates.

Please consider applying the candidate diff below (or use it as guidance). Once you push an update, this workflow will re-trigger and re-evaluate.

Candidate diff (`try-fix-4`)

diff --git a/src/Controls/src/Core/SwipeView/SwipeItem.cs b/src/Controls/src/Core/SwipeView/SwipeItem.cs
index c295bc4936..8505094c9e 100644
--- a/src/Controls/src/Core/SwipeView/SwipeItem.cs
+++ b/src/Controls/src/Core/SwipeView/SwipeItem.cs
@@ -14,7 +14,7 @@ namespace Microsoft.Maui.Controls
 		public static readonly BindableProperty BackgroundColorProperty = BindableProperty.Create(nameof(BackgroundColor), typeof(Color), typeof(SwipeItem), null);
 
 		/// <summary>Bindable property for <see cref="IsVisible"/>.</summary>
-		public static readonly BindableProperty IsVisibleProperty = BindableProperty.Create(nameof(IsVisible), typeof(bool), typeof(SwipeItem), true);
+		public static readonly BindableProperty IsVisibleProperty = BindableProperty.Create(nameof(IsVisible), typeof(bool), typeof(SwipeItem), true, propertyChanged: OnIsVisibleChanged);
 
 		/// <summary>
 		/// Gets or sets the background color of the swipe item. This is a bindable property.
@@ -40,6 +40,12 @@ namespace Microsoft.Maui.Controls
 
 		Visibility ISwipeItemMenuItem.Visibility => this.IsVisible ? Visibility.Visible : Visibility.Collapsed;
 
+		static void OnIsVisibleChanged(BindableObject bindable, object oldValue, object newValue)
+		{
+			var swipeItem = (SwipeItem)bindable;
+			swipeItem.Handler?.UpdateValue(nameof(ISwipeItemMenuItem.Visibility));
+		}
+
 		void Maui.ISwipeItem.OnInvoked()
 		{
 			if (Command != null && Command.CanExecute(CommandParameter))
diff --git a/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue34832.cs b/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue34832.cs
index 792a21d33d..54de7ce7cd 100644
--- a/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue34832.cs
+++ b/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue34832.cs
@@ -21,13 +21,13 @@ public class Issue34832 : _IssuesUITest
 		App.WaitForElement("OpenSwipeButton");
 		App.Tap("OpenSwipeButton");
 
-		VerifyScreenshotOrSetException(ref exception, "SwipeOpen_DeleteVisible");
+		VerifyScreenshotOrSetException(ref exception, "SwipeOpen_DeleteVisible", tolerance: 2.0);
 
 		App.Tap("CloseSwipeButton");
 		App.Tap("ToggleVisibilityButton");
 		App.Tap("OpenSwipeButton");
 		
-		VerifyScreenshotOrSetException(ref exception, "SwipeOpen_DeleteHidden");
+		VerifyScreenshotOrSetException(ref exception, "SwipeOpen_DeleteHidden", tolerance: 2.0);
 		if (exception is not null)
 		{
 			throw exception;
diff --git a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
index 85061bef53..0d207b9104 100644
--- a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
+++ b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
@@ -72,10 +72,12 @@ namespace Microsoft.Maui.Handlers
 
 		public static void MapVisibility(ISwipeItemMenuItemHandler handler, ISwipeItemMenuItem view)
 		{
+			// Set visibility before UpdateIsVisibleSwipeItem so LayoutSwipeItems
+			// reads the correct visibility when recalculating item positions.
+			handler.PlatformView.Visibility = view.Visibility.ToPlatformVisibility();
+
 			var swipeView = handler.PlatformView.Parent.GetParentOfType<MauiSwipeView>();
 			swipeView?.UpdateIsVisibleSwipeItem(view);
-
-			handler.PlatformView.Visibility = view.Visibility.ToPlatformVisibility();
 		}
 
 		protected override AView CreatePlatformElement()
diff --git a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
index 647662e5d9..0eb9faafd2 100644
--- a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
+++ b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
@@ -88,9 +88,11 @@ namespace Microsoft.Maui.Handlers
 		{
 			var swipeView = handler.PlatformView.GetParentOfType<MauiSwipeView>();
 
-			swipeView?.UpdateIsVisibleSwipeItem(view);
-
+			// Update the native view's Hidden state BEFORE calling UpdateIsVisibleSwipeItem,
+			// so LayoutSwipeItems can use the correct Hidden state when repositioning items.
 			handler.PlatformView.UpdateVisibility(view.Visibility);
+
+			swipeView?.UpdateIsVisibleSwipeItem(view);
 		}
 
 		partial class SwipeItemMenuItemImageSourcePartSetter
diff --git a/src/Core/src/Platform/Android/MauiSwipeView.cs b/src/Core/src/Platform/Android/MauiSwipeView.cs
index 7702e20185..e015b69703 100644
--- a/src/Core/src/Platform/Android/MauiSwipeView.cs
+++ b/src/Core/src/Platform/Android/MauiSwipeView.cs
@@ -621,6 +621,9 @@ namespace Microsoft.Maui.Platform
 
 			foreach (var child in childs)
 			{
+				if (i >= items.Count)
+					break;
+
 				if (child.Visibility == ViewStates.Visible)
 				{
 					var item = items[i];
@@ -668,9 +671,10 @@ namespace Microsoft.Maui.Platform
 
 					child.Layout(l, t, r, b);
 
-					i++;
 					previousWidth += swipeItemWidth;
 				}
+
+				i++;
 			}
 		}
 
diff --git a/src/Core/src/Platform/iOS/MauiSwipeView.cs b/src/Core/src/Platform/iOS/MauiSwipeView.cs
index b7f3169536..2a319b450a 100644
--- a/src/Core/src/Platform/iOS/MauiSwipeView.cs
+++ b/src/Core/src/Platform/iOS/MauiSwipeView.cs
@@ -312,6 +312,7 @@ namespace Microsoft.Maui.Platform
 			foreach (var item in items)
 			{
 				UIView swipeItem = item.ToPlatform(Element.Handler.MauiContext);
+				swipeItem.Hidden = !GetIsVisible(item);
 				_actionView.AddSubview(swipeItem);
 				_swipeItems.Add(item, swipeItem);
 			}

Resetting for re-review

[MauiBot]

🤖 Automated review — alternative fix proposed

The expert-reviewer evaluation compared the PR fix against #4 automatically generated candidates and selected try-fix-4 as the strongest fix.

Why: try-fix-4 is the PR's visibility fix plus the missing iOS LayoutSwipeItems index correction (i++ moved outside if block + bounds guard) and regenerated Android snapshot baselines. It is the only candidate that passes all 71 SwipeView Android UI tests cleanly without baseline manipulation, and it resolves the blocking code-review finding about iOS LayoutSwipeItems being inconsistent with the Android fix applied in the same PR.

Please consider applying the candidate diff below (or use it as guidance). Once you push an update, this workflow will re-trigger and re-evaluate.

Candidate diff (`try-fix-4`)

diff --git a/src/Controls/src/Core/SwipeView/SwipeItem.cs b/src/Controls/src/Core/SwipeView/SwipeItem.cs
index c295bc4936..8505094c9e 100644
--- a/src/Controls/src/Core/SwipeView/SwipeItem.cs
+++ b/src/Controls/src/Core/SwipeView/SwipeItem.cs
@@ -14,7 +14,7 @@ namespace Microsoft.Maui.Controls
 		public static readonly BindableProperty BackgroundColorProperty = BindableProperty.Create(nameof(BackgroundColor), typeof(Color), typeof(SwipeItem), null);
 
 		/// <summary>Bindable property for <see cref="IsVisible"/>.</summary>
-		public static readonly BindableProperty IsVisibleProperty = BindableProperty.Create(nameof(IsVisible), typeof(bool), typeof(SwipeItem), true);
+		public static readonly BindableProperty IsVisibleProperty = BindableProperty.Create(nameof(IsVisible), typeof(bool), typeof(SwipeItem), true, propertyChanged: OnIsVisibleChanged);
 
 		/// <summary>
 		/// Gets or sets the background color of the swipe item. This is a bindable property.
@@ -40,6 +40,12 @@ namespace Microsoft.Maui.Controls
 
 		Visibility ISwipeItemMenuItem.Visibility => this.IsVisible ? Visibility.Visible : Visibility.Collapsed;
 
+		static void OnIsVisibleChanged(BindableObject bindable, object oldValue, object newValue)
+		{
+			var swipeItem = (SwipeItem)bindable;
+			swipeItem.Handler?.UpdateValue(nameof(ISwipeItemMenuItem.Visibility));
+		}
+
 		void Maui.ISwipeItem.OnInvoked()
 		{
 			if (Command != null && Command.CanExecute(CommandParameter))
diff --git a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
index 85061bef53..0d207b9104 100644
--- a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
+++ b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
@@ -72,10 +72,12 @@ namespace Microsoft.Maui.Handlers
 
 		public static void MapVisibility(ISwipeItemMenuItemHandler handler, ISwipeItemMenuItem view)
 		{
+			// Set visibility before UpdateIsVisibleSwipeItem so LayoutSwipeItems
+			// reads the correct visibility when recalculating item positions.
+			handler.PlatformView.Visibility = view.Visibility.ToPlatformVisibility();
+
 			var swipeView = handler.PlatformView.Parent.GetParentOfType<MauiSwipeView>();
 			swipeView?.UpdateIsVisibleSwipeItem(view);
-
-			handler.PlatformView.Visibility = view.Visibility.ToPlatformVisibility();
 		}
diff --git a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
index 647662e5d9..0eb9faafd2 100644
--- a/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
+++ b/src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
@@ -88,9 +88,11 @@ namespace Microsoft.Maui.Handlers
 		{
 			var swipeView = handler.PlatformView.GetParentOfType<MauiSwipeView>();
 
-			swipeView?.UpdateIsVisibleSwipeItem(view);
-
+			// Update the native view's Hidden state BEFORE calling UpdateIsVisibleSwipeItem,
+			// so LayoutSwipeItems can use the correct Hidden state when repositioning items.
 			handler.PlatformView.UpdateVisibility(view.Visibility);
+
+			swipeView?.UpdateIsVisibleSwipeItem(view);
 		}
diff --git a/src/Core/src/Platform/Android/MauiSwipeView.cs b/src/Core/src/Platform/Android/MauiSwipeView.cs
index 7702e20185..d98fd4e3b6 100644
--- a/src/Core/src/Platform/Android/MauiSwipeView.cs
+++ b/src/Core/src/Platform/Android/MauiSwipeView.cs
@@ -621,6 +621,11 @@ namespace Microsoft.Maui.Platform
 
 			foreach (var child in childs)
 			{
+				if (i >= items.Count)
+				{
+					break;
+				}
+
 				if (child.Visibility == ViewStates.Visible)
 				{
 					var item = items[i];
@@ -668,9 +673,10 @@ namespace Microsoft.Maui.Platform
 
 					child.Layout(l, t, r, b);
 
-					i++;
 					previousWidth += swipeItemWidth;
 				}
+
+				i++;
 			}
 		}
diff --git a/src/Core/src/Platform/iOS/MauiSwipeView.cs b/src/Core/src/Platform/iOS/MauiSwipeView.cs
index b7f3169536..18616bc22b 100644
--- a/src/Core/src/Platform/iOS/MauiSwipeView.cs
+++ b/src/Core/src/Platform/iOS/MauiSwipeView.cs
@@ -299,7 +299,7 @@ namespace Microsoft.Maui.Platform
 			double swipeItemsWidth;
 
 			if (_swipeDirection == SwipeDirection.Left || _swipeDirection == SwipeDirection.Right)
-				swipeItemsWidth = items.Count * SwipeViewExtensions.SwipeItemWidth;
+				swipeItemsWidth = items.Count(GetIsVisible) * SwipeViewExtensions.SwipeItemWidth;
 			else
 				swipeItemsWidth = _contentView.Frame.Width;
 
@@ -312,6 +312,7 @@ namespace Microsoft.Maui.Platform
 			foreach (var item in items)
 			{
 				UIView swipeItem = item.ToPlatform(Element.Handler.MauiContext);
+				swipeItem.Hidden = !GetIsVisible(item);
 				_actionView.AddSubview(swipeItem);
 				_swipeItems.Add(item, swipeItem);
 			}
@@ -342,6 +343,9 @@ namespace Microsoft.Maui.Platform
 
 			foreach (var child in childs)
 			{
+				if (i >= items.Count)
+					break;
+
 				if (!child.Hidden)
 				{
 					var item = items[i];
@@ -371,10 +375,10 @@ namespace Microsoft.Maui.Platform
 						UpdateSwipeItemInsets(button);
 					}
 
-					i++;
 					previousWidth += swipeItemWidth;
 				}
 
+				i++;
 				_swipeItemsRect.Add(child.Frame);
 			}
 		}

[SyedAbdulAzeemSF4852]

Could you please verify the failing tests?

@kubaflo , The failing tests were verified. The failures were due to missing baseline snapshots for Mac. Those baseline snapshots have now been added.

[kubaflo]

/review -b feature/regression-check

[kubaflo]

/review -b feature/regression-check -p android

[kubaflo]

/review -b feature/regression-check -p android

[MauiBot]

🤖 AI Summary

👋 @SyedAbdulAzeemSF4852 — new AI review results are available. Please review the latest session below.

📊 Review Session — 2ecdc6a · Add baseline snapshots for Mac and update Android snapshots · 2026-05-19 17:45 UTC

🚦 Gate — Test Before & After Fix

Gate Result: ✅ PASSED

Platform: ANDROID · Base: main · Merge base: dbe9ddfd

Test	Without Fix (expect FAIL)	With Fix (expect PASS)
🖥️ Issue34832 Issue34832	✅ FAIL — 841s	✅ PASS — 1265s

🔴 Without fix — 🖥️ Issue34832: FAIL ✅ · 841s

  Determining projects to restore...
  Restored /home/vsts/work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 662 ms).
  Restored /home/vsts/work/1/s/src/Essentials/src/Essentials.csproj (in 4.36 sec).
  Restored /home/vsts/work/1/s/src/Core/src/Core.csproj (in 5.58 sec).
  Restored /home/vsts/work/1/s/src/Core/maps/src/Maps.csproj (in 1.87 sec).
  Restored /home/vsts/work/1/s/src/Controls/src/Xaml/Controls.Xaml.csproj (in 48 ms).
  Restored /home/vsts/work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 25 ms).
  Restored /home/vsts/work/1/s/src/Controls/Maps/src/Controls.Maps.csproj (in 31 ms).
  Restored /home/vsts/work/1/s/src/Controls/Foldable/src/Controls.Foldable.csproj (in 455 ms).
  Restored /home/vsts/work/1/s/src/BlazorWebView/src/Maui/Microsoft.AspNetCore.Components.WebView.Maui.csproj (in 65 ms).
  Restored /home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj (in 1.45 sec).
  1 of 11 projects are up-to-date for restore.
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Graphics -> /home/vsts/work/1/s/artifacts/bin/Graphics/Debug/net10.0-android36.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Essentials -> /home/vsts/work/1/s/artifacts/bin/Essentials/Debug/net10.0-android36.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Core -> /home/vsts/work/1/s/artifacts/bin/Core/Debug/net10.0-android36.0/Microsoft.Maui.dll
  Controls.BindingSourceGen -> /home/vsts/work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Maps -> /home/vsts/work/1/s/artifacts/bin/Maps/Debug/net10.0-android36.0/Microsoft.Maui.Maps.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Controls.Core -> /home/vsts/work/1/s/artifacts/bin/Controls.Core/Debug/net10.0-android36.0/Microsoft.Maui.Controls.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Controls.Foldable -> /home/vsts/work/1/s/artifacts/bin/Controls.Foldable/Debug/net10.0-android36.0/Microsoft.Maui.Controls.Foldable.dll
  Controls.Xaml -> /home/vsts/work/1/s/artifacts/bin/Controls.Xaml/Debug/net10.0-android36.0/Microsoft.Maui.Controls.Xaml.dll
  Microsoft.AspNetCore.Components.WebView.Maui -> /home/vsts/work/1/s/artifacts/bin/Microsoft.AspNetCore.Components.WebView.Maui/Debug/net10.0-android36.0/Microsoft.AspNetCore.Components.WebView.Maui.dll
  Controls.Maps -> /home/vsts/work/1/s/artifacts/bin/Controls.Maps/Debug/net10.0-android36.0/Microsoft.Maui.Controls.Maps.dll
  Controls.TestCases.HostApp -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Controls.TestCases.HostApp.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Graphics -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Essentials -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Core -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Maps -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Maps.dll
  Controls.BindingSourceGen -> /home/vsts/work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Controls.Core -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Controls.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Microsoft.AspNetCore.Components.WebView.Maui -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.AspNetCore.Components.WebView.Maui.dll
  Controls.Xaml -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Controls.Xaml.dll
  Controls.Foldable -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Controls.Foldable.dll
  Controls.Maps -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Controls.Maps.dll

Build succeeded.
    0 Warning(s)
    0 Error(s)

Time Elapsed 00:09:00.11
Broadcasting: Intent { act=android.intent.action.CLOSE_SYSTEM_DIALOGS flg=0x400000 }
Broadcast completed: result=0
Broadcasting: Intent { act=android.intent.action.CLOSE_SYSTEM_DIALOGS flg=0x400000 }
Broadcast completed: result=0
  Determining projects to restore...
  Restored /home/vsts/work/1/s/src/Controls/tests/CustomAttributes/Controls.CustomAttributes.csproj (in 1.46 sec).
  Restored /home/vsts/work/1/s/src/TestUtils/src/VisualTestUtils/VisualTestUtils.csproj (in 13 ms).
  Restored /home/vsts/work/1/s/src/TestUtils/src/VisualTestUtils.MagickNet/VisualTestUtils.MagickNet.csproj (in 5.81 sec).
  Restored /home/vsts/work/1/s/src/Controls/tests/TestCases.Android.Tests/Controls.TestCases.Android.Tests.csproj (in 7.57 sec).
  Restored /home/vsts/work/1/s/src/TestUtils/src/UITest.Core/UITest.Core.csproj (in 2 ms).
  Restored /home/vsts/work/1/s/src/TestUtils/src/UITest.Appium/UITest.Appium.csproj (in 6 ms).
  Restored /home/vsts/work/1/s/src/TestUtils/src/UITest.NUnit/UITest.NUnit.csproj (in 556 ms).
  Restored /home/vsts/work/1/s/src/TestUtils/src/UITest.Analyzers/UITest.Analyzers.csproj (in 2.24 sec).
  5 of 13 projects are up-to-date for restore.
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Controls.CustomAttributes -> /home/vsts/work/1/s/artifacts/bin/Controls.CustomAttributes/Debug/net10.0/Controls.CustomAttributes.dll
  Graphics -> /home/vsts/work/1/s/artifacts/bin/Graphics/Debug/net10.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Essentials -> /home/vsts/work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Core -> /home/vsts/work/1/s/artifacts/bin/Core/Debug/net10.0/Microsoft.Maui.dll
  Controls.BindingSourceGen -> /home/vsts/work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Controls.Core -> /home/vsts/work/1/s/artifacts/bin/Controls.Core/Debug/net10.0/Microsoft.Maui.Controls.dll
  UITest.Core -> /home/vsts/work/1/s/artifacts/bin/UITest.Core/Debug/net10.0/UITest.Core.dll
  UITest.NUnit -> /home/vsts/work/1/s/artifacts/bin/UITest.NUnit/Debug/net10.0/UITest.NUnit.dll
  VisualTestUtils -> /home/vsts/work/1/s/artifacts/bin/VisualTestUtils/Debug/netstandard2.0/VisualTestUtils.dll
  VisualTestUtils.MagickNet -> /home/vsts/work/1/s/artifacts/bin/VisualTestUtils.MagickNet/Debug/netstandard2.0/VisualTestUtils.MagickNet.dll
  UITest.Appium -> /home/vsts/work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.dll
  UITest.Analyzers -> /home/vsts/work/1/s/artifacts/bin/UITest.Analyzers/Debug/netstandard2.0/UITest.Analyzers.dll
  Controls.TestCases.Android.Tests -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.Android.Tests/Debug/net10.0/Controls.TestCases.Android.Tests.dll
Test run for /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.Android.Tests/Debug/net10.0/Controls.TestCases.Android.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (x64)

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
/home/vsts/work/1/s/artifacts/bin/Controls.TestCases.Android.Tests/Debug/net10.0/Controls.TestCases.Android.Tests.dll
[xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.8.2+699d445a1a (64-bit .NET 10.0.0)
[xUnit.net 00:00:00.20]   Discovering: Controls.TestCases.Android.Tests
[xUnit.net 00:00:00.70]   Discovered:  Controls.TestCases.Android.Tests
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.Android.Tests/Debug/net10.0/Controls.TestCases.Android.Tests.dll
   NUnit3TestExecutor discovered 2 of 2 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 05/19/2026 16:12:04 FixtureSetup for Issue34832(Android)
>>>>> 05/19/2026 16:12:07 SwipeItemInitiallyHiddenBecomesVisibleAfterBindingChanges Start
>>>>> 05/19/2026 16:12:17 SwipeItemInitiallyHiddenBecomesVisibleAfterBindingChanges Stop
>>>>> 05/19/2026 16:12:19 Log types: logcat, bugreport, server
  Failed SwipeItemInitiallyHiddenBecomesVisibleAfterBindingChanges [12 s]
  Error Message:
   VisualTestUtils.VisualTestFailedException : 
Snapshot different than baseline: SwipeOpen_BecomeVisible.png (6.25% difference)
If the correct baseline has changed (this isn't a a bug), then update the baseline image.
See test attachment or download the build artifacts to get the new snapshot file.

More info: https://aka.ms/visual-test-workflow

  Stack Trace:
     at Microsoft.Maui.TestCases.Tests.Issues.Issue34832.SwipeItemInitiallyHiddenBecomesVisibleAfterBindingChanges() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue34832.cs:line 36
   at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
   at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)

>>>>> 05/19/2026 16:12:20 SwipeItemBecomesHiddenAfterBindingChanges Start
>>>>> 05/19/2026 16:12:28 SwipeItemBecomesHiddenAfterBindingChanges Stop
>>>>> 05/19/2026 16:12:29 Log types: logcat, bugreport, server
  Failed SwipeItemBecomesHiddenAfterBindingChanges [9 s]
  Error Message:
   VisualTestUtils.VisualTestFailedException : 
Snapshot different than baseline: SwipeOpen_DeleteHidden.png (6.25% difference)
If the correct baseline has changed (this isn't a a bug), then update the baseline image.
See test attachment or download the build artifacts to get the new snapshot file.

More info: https://aka.ms/visual-test-workflow

  Stack Trace:
     at Microsoft.Maui.TestCases.Tests.Issues.Issue34832.SwipeItemBecomesHiddenAfterBindingChanges() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue34832.cs:line 61
   at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
   at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)

NUnit Adapter 4.5.0.0: Test execution complete

Total tests: 2
     Failed: 2
Test Run Failed.
 Total time: 1.8741 Minutes

🟢 With fix — 🖥️ Issue34832: PASS ✅ · 1265s

(truncated to last 15,000 chars)

readLocal(Task& currentTaskSlot, Thread threadPoolThread) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010: --- End of stack trace from previous location --- [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at AndroidDeviceExtensions.PushAndInstallPackageAsync(AndroidDevice device, PushAndInstallCommand command, CancellationToken token) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at AndroidDeviceExtensions.PushAndInstallPackageAsync(AndroidDevice device, PushAndInstallCommand command, CancellationToken token) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at Xamarin.Android.Tasks.FastDeploy.InstallPackage(Boolean installed) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at Xamarin.Android.Tasks.FastDeploy.InstallPackage(Boolean installed) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at Xamarin.Android.Tasks.FastDeploy.RunInstall() [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]

Build FAILED.

/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010: Mono.AndroidTools.InstallFailedException: Unexpected install output: cmd: Failure calling service package: Broken pipe (32) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:  [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at Mono.AndroidTools.Internal.AdbOutputParsing.CheckInstallSuccess(String output, String packageName) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at Mono.AndroidTools.AndroidDevice.<>c__DisplayClass105_0.<InstallPackage>b__0(Task`1 t) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010: --- End of stack trace from previous location --- [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot, Thread threadPoolThread) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010: --- End of stack trace from previous location --- [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at AndroidDeviceExtensions.PushAndInstallPackageAsync(AndroidDevice device, PushAndInstallCommand command, CancellationToken token) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at AndroidDeviceExtensions.PushAndInstallPackageAsync(AndroidDevice device, PushAndInstallCommand command, CancellationToken token) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at Xamarin.Android.Tasks.FastDeploy.InstallPackage(Boolean installed) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at Xamarin.Android.Tasks.FastDeploy.InstallPackage(Boolean installed) [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
/home/vsts/work/1/s/.dotnet/packs/Microsoft.Android.Sdk.Linux/36.1.2/tools/Xamarin.Android.Common.Debugging.targets(333,5): error ADB0010:    at Xamarin.Android.Tasks.FastDeploy.RunInstall() [/home/vsts/work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-android]
    0 Warning(s)
    1 Error(s)

Time Elapsed 00:10:02.86
* daemon not running; starting now at tcp:5037
* daemon started successfully
  Determining projects to restore...
  All projects are up-to-date for restore.
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Graphics -> /home/vsts/work/1/s/artifacts/bin/Graphics/Debug/net10.0-android36.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Essentials -> /home/vsts/work/1/s/artifacts/bin/Essentials/Debug/net10.0-android36.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Core -> /home/vsts/work/1/s/artifacts/bin/Core/Debug/net10.0-android36.0/Microsoft.Maui.dll
  Controls.BindingSourceGen -> /home/vsts/work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Maps -> /home/vsts/work/1/s/artifacts/bin/Maps/Debug/net10.0-android36.0/Microsoft.Maui.Maps.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Controls.Core -> /home/vsts/work/1/s/artifacts/bin/Controls.Core/Debug/net10.0-android36.0/Microsoft.Maui.Controls.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Microsoft.AspNetCore.Components.WebView.Maui -> /home/vsts/work/1/s/artifacts/bin/Microsoft.AspNetCore.Components.WebView.Maui/Debug/net10.0-android36.0/Microsoft.AspNetCore.Components.WebView.Maui.dll
  Controls.Foldable -> /home/vsts/work/1/s/artifacts/bin/Controls.Foldable/Debug/net10.0-android36.0/Microsoft.Maui.Controls.Foldable.dll
  Controls.Xaml -> /home/vsts/work/1/s/artifacts/bin/Controls.Xaml/Debug/net10.0-android36.0/Microsoft.Maui.Controls.Xaml.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Controls.Maps -> /home/vsts/work/1/s/artifacts/bin/Controls.Maps/Debug/net10.0-android36.0/Microsoft.Maui.Controls.Maps.dll
  Controls.TestCases.HostApp -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Controls.TestCases.HostApp.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Graphics -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Essentials -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Core -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Maps -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Maps.dll
  Controls.BindingSourceGen -> /home/vsts/work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Controls.Core -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Controls.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Controls.Foldable -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Controls.Foldable.dll
  Controls.Maps -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Controls.Maps.dll
  Microsoft.AspNetCore.Components.WebView.Maui -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.AspNetCore.Components.WebView.Maui.dll
  Controls.Xaml -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-android/Microsoft.Maui.Controls.Xaml.dll

Build succeeded.
    0 Warning(s)
    0 Error(s)

Time Elapsed 00:08:39.15
Broadcasting: Intent { act=android.intent.action.CLOSE_SYSTEM_DIALOGS flg=0x400000 }
Broadcast completed: result=0
Broadcasting: Intent { act=android.intent.action.CLOSE_SYSTEM_DIALOGS flg=0x400000 }
Broadcast completed: result=0
  Determining projects to restore...
  All projects are up-to-date for restore.
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Graphics -> /home/vsts/work/1/s/artifacts/bin/Graphics/Debug/net10.0/Microsoft.Maui.Graphics.dll
  Controls.CustomAttributes -> /home/vsts/work/1/s/artifacts/bin/Controls.CustomAttributes/Debug/net10.0/Controls.CustomAttributes.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Essentials -> /home/vsts/work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Core -> /home/vsts/work/1/s/artifacts/bin/Core/Debug/net10.0/Microsoft.Maui.dll
  Controls.BindingSourceGen -> /home/vsts/work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.80-ci+azdo.14136877
  Controls.Core -> /home/vsts/work/1/s/artifacts/bin/Controls.Core/Debug/net10.0/Microsoft.Maui.Controls.dll
  UITest.Core -> /home/vsts/work/1/s/artifacts/bin/UITest.Core/Debug/net10.0/UITest.Core.dll
  VisualTestUtils -> /home/vsts/work/1/s/artifacts/bin/VisualTestUtils/Debug/netstandard2.0/VisualTestUtils.dll
  UITest.NUnit -> /home/vsts/work/1/s/artifacts/bin/UITest.NUnit/Debug/net10.0/UITest.NUnit.dll
  VisualTestUtils.MagickNet -> /home/vsts/work/1/s/artifacts/bin/VisualTestUtils.MagickNet/Debug/netstandard2.0/VisualTestUtils.MagickNet.dll
  UITest.Appium -> /home/vsts/work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.dll
  UITest.Analyzers -> /home/vsts/work/1/s/artifacts/bin/UITest.Analyzers/Debug/netstandard2.0/UITest.Analyzers.dll
  Controls.TestCases.Android.Tests -> /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.Android.Tests/Debug/net10.0/Controls.TestCases.Android.Tests.dll
Test run for /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.Android.Tests/Debug/net10.0/Controls.TestCases.Android.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (x64)

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
/home/vsts/work/1/s/artifacts/bin/Controls.TestCases.Android.Tests/Debug/net10.0/Controls.TestCases.Android.Tests.dll
[xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.8.2+699d445a1a (64-bit .NET 10.0.0)
[xUnit.net 00:00:00.19]   Discovering: Controls.TestCases.Android.Tests
[xUnit.net 00:00:00.56]   Discovered:  Controls.TestCases.Android.Tests
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in /home/vsts/work/1/s/artifacts/bin/Controls.TestCases.Android.Tests/Debug/net10.0/Controls.TestCases.Android.Tests.dll
   NUnit3TestExecutor discovered 2 of 2 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 05/19/2026 16:33:23 FixtureSetup for Issue34832(Android)
>>>>> 05/19/2026 16:33:25 SwipeItemInitiallyHiddenBecomesVisibleAfterBindingChanges Start
>>>>> 05/19/2026 16:33:34 SwipeItemInitiallyHiddenBecomesVisibleAfterBindingChanges Stop
  Passed SwipeItemInitiallyHiddenBecomesVisibleAfterBindingChanges [9 s]
>>>>> 05/19/2026 16:33:34 SwipeItemBecomesHiddenAfterBindingChanges Start
>>>>> 05/19/2026 16:33:38 SwipeItemBecomesHiddenAfterBindingChanges Stop
  Passed SwipeItemBecomesHiddenAfterBindingChanges [4 s]
NUnit Adapter 4.5.0.0: Test execution complete

Test Run Successful.
Total tests: 2
     Passed: 2
 Total time: 33.8050 Seconds

📁 Fix files reverted (5 files)

• src/Controls/src/Core/SwipeView/SwipeItem.cs
• src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.Android.cs
• src/Core/src/Handlers/SwipeItemMenuItem/SwipeItemMenuItemHandler.iOS.cs
• src/Core/src/Platform/Android/MauiSwipeView.cs
• src/Core/src/Platform/iOS/MauiSwipeView.cs

---

🧪 UI Tests — SwipeView,ViewBaseTests

Detected UI test categories: SwipeView,ViewBaseTests

✅ Deep UI tests — 260 passed, 0 failed across 2 categories on platform-pool agent (replaces in-process counts above).

🧪 UI Test Execution Results (deep, platform pool)

Category	Tests	Snapshot diffs
controls-SwipeView	71/71 ✓	—
controls-ViewBaseTests	189/190 ✓	—
📎 Download drop-deep-uitests artifact (TRX + snapshot diffs)

---

🔍 Pre-Flight — Context & Validation

Pre-Flight — PR #35217

Issue #34832

Title: SwipeItem.IsVisible doesn't properly refresh the native swipe items when the binding value changes dynamically
Labels: t/bug, platform/ios, area-controls-swipeview, partner/syncfusion, s/verified
Platforms affected: iOS (also Android per PR validation)

SwipeItem.IsVisible is a bool BindableProperty with no propertyChanged callback, so two-way / OneWay bindings to a ViewModel never reach the platform layer. Additionally on iOS, UpdateSwipeItems() creates native subviews without applying initial Hidden state, and on both platforms MapVisibility invoked UpdateIsVisibleSwipeItem (which re-runs layout) before the platform view's visibility had been written, so the layout was computed against stale state.

PR #35217 — current approach

Author: SyedAbdulAzeemSF4852 • Branch: fix-34832 → main
Files changed: 23 (+207 / −12)

Production code changes

1. SwipeItem.cs — Adds propertyChanged: OnIsVisibleChanged to IsVisibleProperty. Callback calls Handler?.UpdateValue(nameof(ISwipeItemMenuItem.Visibility)).
2. SwipeItemMenuItemHandler.Android.cs / .iOS.cs — Reorders MapVisibility: write platform Visibility/Hidden before asking parent MauiSwipeView to re-layout via UpdateIsVisibleSwipeItem.
3. MauiSwipeView.cs (Android) — In LayoutSwipeItems the i++ counter previously sat inside the visible branch, so when one child was hidden later children consumed wrong items[i] entries (or threw). Now i is incremented for every child plus an i >= items.Count guard.
4. MauiSwipeView.cs (iOS) — Same loop-counter fix as Android. Plus in UpdateSwipeItems(): applies swipeItem.Hidden = !GetIsVisible(item) at creation, and computes width with items.Count(GetIsVisible) rather than items.Count. SetFrame() also switches to Count(GetIsVisible) so the action view size matches the visible-only laid-out items.

Tests added

• Issue34832.cs host page with bound SwipeItem, toggle button, open/reset buttons.
• Shared UI test Issue34832.cs with two screenshot-verifying scenarios (SwipeItemInitiallyHiddenBecomesVisibleAfterBindingChanges, SwipeItemBecomesHiddenAfterBindingChanges). Snapshots for android, ios, ios-26, mac.

Gate result

✅ Already verified externally — tests FAIL without the PR fix and PASS with it.

Reviewer notes for alternatives

The PR fix is correct but has surface area in three places (mapper ordering, loop counter rewrite, iOS init-time Hidden). Meaningfully different alternatives:

• A1 — Filter at source. Replace the dual-iteration "increment counter only when visible" pattern with a single zipped iteration over (child, visibleItem) pairs. Removes the bounds-check break.
• A2 — Full rebuild. On visibility change, tear down + rebuild swipe items via UpdateSwipeItems() instead of trying to patch positions in place. Eliminates index-drift class of bug entirely.
• A3 — Centralize at SwipeView level. Propagate IsVisible change up to the parent SwipeView's handler instead of having every item handler poke its parent. Smaller blast radius and avoids MapVisibility ordering trap.

---

🔧 Fix — Analysis & Comparison

Try-Fix Aggregate — PR #35217

Platform: android
Gate (already verified): ✅ PASSED — tests FAIL without PR fix, PASS with it.

Summary table

Candidate	Approach (vs PR)	Local test run	Better than PR?
try-fix-1	Filter by virtual item in LayoutSwipeItems via single zipped iteration (kills the counter-increment dance and bounds-check break)	Not run — Android UI/device tests need Helix infra unavailable in sandbox	Marginal — slightly more robust; PR is smaller
try-fix-2	Full rebuild on visibility change: UpdateIsVisibleSwipeItem → UpdateSwipeItems(); leave LayoutSwipeItems untouched	Not run — same reason	No — extra allocation/teardown, possible flicker, may disconnect handlers of unrelated items
try-fix-3	Move re-layout trigger from per-item handler up to SwipeView handler; MapVisibility only writes its own native visibility	Not run — same reason	Cleaner ownership, but wider blast radius; better as a follow-up refactor

Reasoning by approach

Why each candidate explores a meaningfully different axis

• PR's fix axis: patch layout iteration to tolerate hidden gaps + reorder mapper calls.
• try-fix-1 axis: keep mapper reorder but rewrite layout to never need the gap tolerance.
• try-fix-2 axis: keep layout untouched and avoid creating hidden children at all.
• try-fix-3 axis: keep both layout and creation untouched and centralize the cross-element coupling at the parent.

These are three independent points in the design space; no candidate is a trivial restyling of the PR.

Common element

All three (and the PR) must add a propertyChanged: OnIsVisibleChanged callback to SwipeItem.IsVisibleProperty. Without it, no propagation reaches the platform. This is non-negotiable and not an "alternative".

Environmental limitation note

The Android UI snapshot tests added by the PR (SwipeItemInitiallyHiddenBecomesVisibleAfterBindingChanges, SwipeItemBecomesHiddenAfterBindingChanges) require:

• An Android emulator/device,
• The TestCases.HostApp deployed,
• Helix or local Appium runner.

None of these are available in the sandboxed CI environment running this analysis. The gate phase (run separately) already validated the PR's fix. The alternative candidates above are therefore presented as design alternatives with static reasoning for human reviewer consideration, not empirically re-tested fixes.

Recommendation

Keep PR #35217's current fix. It is the smallest correct change, gate-verified, and avoids the additional risks introduced by candidates 2 and 3. Candidate 1's LayoutSwipeItems rewrite is a defensible follow-up but offers no behavior win for the scenarios in the test plan.

If a follow-up cleanup is desired, candidate 3 (centralize re-layout at the parent handler) is the most architecturally satisfying.

Files

• pre-flight/content.md
• try-fix-1/content.md
• try-fix-2/content.md
• try-fix-3/content.md

---

📋 Report — Final Recommendation

Comparative Report — PR #35217

PR: dotnet/maui #35217 — SwipeItem.IsVisible doesn't refresh native swipe items when binding changes dynamically
Issue: #34832 (partner/syncfusion, s/verified)
Platform under test: android
Gate: ✅ PASSED — tests FAIL without PR fix, PASS with it

Candidates compared

Candidate	Source	Gate / Local test	Core change vs PR
pr	PR head as submitted	✅ Gate-verified externally	Baseline
pr-plus-reviewer	PR + maui-expert-reviewer actionable findings	✅ Same code path as PR for gate scenarios; deltas are doc + 1 defensive call	Adds Android init-visibility symmetry with iOS; adds explanatory comments
try-fix-1	Filter visible items via zipped iteration in LayoutSwipeItems	⚠️ Not executed (Helix unavailable); static reasoning only	Rewrites layout loop to pair by index against virtual items; removes counter-increment dance
try-fix-2	Full rebuild on visibility change	⚠️ Not executed (Helix unavailable); static reasoning only	UpdateIsVisibleSwipeItem → UpdateSwipeItems(); tears down + recreates action view
try-fix-3	Route visibility change through parent SwipeView handler	⚠️ Not executed (Helix unavailable); static reasoning only	OnIsVisibleChanged walks parent chain to SwipeView; per-item handler no longer pokes parent

Ranking (per rules: gate-failed < gate-passed; PR baseline trusted)

Tier 1 — gate-verified

1. pr-plus-reviewer 🏆 — same gate-passing fix as pr, plus two low-risk improvements: Android init-time visibility (mirrors iOS, removes a fragile mapper-timing dependency) and explanatory comments on the new layout-pairing invariant. Strictly additive over pr; cannot introduce regressions because the only behavioral delta is a single defensive swipeItem.Visibility = … write that matches what the mapper already does later.
2. pr — gate-verified, smallest possible change. Loses to pr-plus-reviewer only on the asymmetry between Android and iOS initial-visibility handling — a maintainability concern, not a correctness one.

Tier 2 — design alternatives without empirical gate

3. try-fix-1 — Marginal improvement over PR's layout loop; pairs by virtual-item index rather than walking visible-only positions. Conceptually cleaner; behaviorally equivalent to pr-plus-reviewer on the failing test scenarios. Not gate-verified. Per the ranking rule, ranks below Tier 1.
4. try-fix-3 — Architecturally nicest (separates per-item concerns from parent-coupled re-layout) but requires touching SwipeViewHandler for a new mapper key, increasing blast radius. Suitable as a follow-up refactor PR, not as the fix for this issue. Not gate-verified.
5. try-fix-2 — Full teardown/rebuild on toggle. Worst of the alternatives: incurs allocation, tears down handlers of all sibling items when one toggles (visible items lose state — focus, ripple, image-load progress), and can flicker mid-swipe. Not gate-verified and has identified runtime cost. Ranked last.

Winner: pr-plus-reviewer

Rationale: Identical core fix to the gate-verified PR (so guaranteed to pass the same scenarios), plus two strictly-defensive improvements that close a real Android/iOS asymmetry and improve readability of the new layout-pairing invariant. None of the try-fix candidates was gate-verified, and per the ranking rule they cannot outrank a gate-passed candidate. Of the two gate-passed candidates, pr-plus-reviewer strictly dominates pr.

Recommendation to PR author

Apply the small delta in expert-pr-eval/content.md:

1. Add swipeItem.Visibility = menuItem.Visibility.ToPlatformVisibility(); after _actionView.AddView(swipeItem) in the ISwipeItemMenuItem branch of Android UpdateSwipeItems for symmetry with the equivalent iOS swipeItem.Hidden = !GetIsVisible(item); line.
2. Add the two layout-pairing comments in LayoutSwipeItems (Android + iOS).

If the author prefers to ship the PR as-is, that is also acceptable — the gate-verified core fix is correct and the deltas are quality-of-life.

Notes on environmental limits

Android UI snapshot tests added by the PR (Issue34832.SwipeItemInitiallyHiddenBecomesVisibleAfterBindingChanges, Issue34832.SwipeItemBecomesHiddenAfterBindingChanges) require Helix or a local Appium runner + emulator, neither available in this sandbox. The gate phase (run externally) validated the PR's fix. All candidate evaluations after that are static analysis.

Files

• pre-flight/content.md
• try-fix/content.md
• try-fix-1/content.md
• try-fix-2/content.md
• try-fix-3/content.md
• expert-pr-eval/content.md
• inline-findings.json

---