[Android/iOS] Fix IsEnabled=False on CollectionView not working - #27749
Conversation
| [Controls.ItemsView.IsVisibleProperty.PropertyName] = MapIsVisible, | ||
| [Controls.ItemsView.ItemsUpdatingScrollModeProperty.PropertyName] = MapItemsUpdatingScrollMode | ||
| [Controls.ItemsView.ItemsUpdatingScrollModeProperty.PropertyName] = MapItemsUpdatingScrollMode, | ||
| #if IOS || ANDROID |
There was a problem hiding this comment.
The behavior is the expected on Catalyst and Windows?
There was a problem hiding this comment.
@jsuarezruiz ,
Yes, this behavior is expected on Windows. With the current iOS changes, it is now applicable to Mac as well
|
|
||
| public override bool OnTouchEvent(MotionEvent e) | ||
| { | ||
| if (!Enabled) |
There was a problem hiding this comment.
I think at this point should also take into account the InputTransparent property.
There was a problem hiding this comment.
@jsuarezruiz,
Without my changes, InputTransparent is already working fine for CollectionView. Let me know if any additional modifications are needed.
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/rebase |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
@devanathan-vaithiyanathan Could you rebase and fix the conflict? Thanks in advance. |
16fcd13 to
8ba7f78
Compare
8ba7f78 to
b540282
Compare
There was a problem hiding this comment.
Pull request overview
This PR fixes a bug where CollectionView items could be selected even when IsEnabled was set to false. The fix implements proper enabled state handling on both iOS and Android platforms by disabling user interaction at the platform control level and adding checks in selection logic.
Key Changes
- iOS: Disables
UserInteractionEnabledon the UICollectionView whenIsEnabledis false - Android: Overrides
OnTouchEventin MauiRecyclerView to block touch events and adds a check in SelectableItemsViewAdapter to prevent selection when disabled - Adds UI test to verify the disabled state prevents selection and enabled state allows selection
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20274.cs | NUnit test that verifies CollectionView selection is blocked when disabled and works when enabled |
| src/Controls/tests/TestCases.HostApp/Issues/Issue20274.xaml.cs | Test page code-behind with CollectionView disabled by default and button to toggle enabled state |
| src/Controls/tests/TestCases.HostApp/Issues/Issue20274.xaml | XAML test page with CollectionView, label, and button for testing IsEnabled behavior |
| src/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cs | Adds MapIsEnabled to Items2 handler (current iOS implementation) to set UserInteractionEnabled |
| src/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cs | Adds MapIsEnabled implementation to Items handler (deprecated iOS implementation) |
| src/Controls/src/Core/Handlers/Items/ItemsViewHandler.cs | Registers MapIsEnabled in property mapper for iOS platform (deprecated Items handler) |
| src/Controls/src/Core/Handlers/Items/Android/MauiRecyclerView.cs | Overrides OnTouchEvent to return false when RecyclerView is disabled |
| src/Controls/src/Core/Handlers/Items/Android/Adapters/SelectableItemsViewAdapter.cs | Adds early return in selection handler when ItemsView.IsEnabled is false |
| [Issue(IssueTracker.Github, 20274, "IsEnabled=False on CollectionView not working", PlatformAffected.All)] | ||
| public partial class Issue20274 : ContentPage | ||
| { | ||
| public ObservableCollection<CollectionViewItem> Items { get; set; } | ||
|
|
||
| public Issue20274() |
There was a problem hiding this comment.
The test files are using issue number 20274 in both the file names and code, but the PR description indicates this PR fixes issue #27770. The issue numbers should match. Consider renaming the test files to Issue27770.cs, Issue27770.xaml, and Issue27770.xaml.cs, and updating the [Issue] attribute to reference the correct issue number 27770.
| [Issue(IssueTracker.Github, 20274, "IsEnabled=False on CollectionView not working", PlatformAffected.All)] | |
| public partial class Issue20274 : ContentPage | |
| { | |
| public ObservableCollection<CollectionViewItem> Items { get; set; } | |
| public Issue20274() | |
| [Issue(IssueTracker.Github, 27770, "IsEnabled=False on CollectionView not working", PlatformAffected.All)] | |
| public partial class Issue27770 : ContentPage | |
| { | |
| public ObservableCollection<CollectionViewItem> Items { get; set; } | |
| public Issue27770() |
| selectedItem.Name = $"{selectedItem.Name} - Selected"; | ||
| this.label.Text = selectedItem.Name; |
There was a problem hiding this comment.
The test logic appears to be incorrect. The test starts with the CollectionView disabled (IsEnabled="False"), taps an item (which should be ignored), then verifies the label shows "Item 1" (the initial state). Then it enables the CollectionView by clicking the button, taps the item again, and expects "Item 1 - Selected". However, there's a bug: on line 28 of the code-behind, when an item is selected, the item's name is mutated to append " - Selected", which means on subsequent selections, it would become "Item 1 - Selected - Selected" and so on. This test only works once. Consider resetting the item name when the CollectionView is re-enabled or using a different mechanism to track selection state.
| selectedItem.Name = $"{selectedItem.Name} - Selected"; | |
| this.label.Text = selectedItem.Name; | |
| this.label.Text = $"{selectedItem.Name} - Selected"; |
| return false; | ||
| return base.OnTouchEvent(e); |
There was a problem hiding this comment.
The indentation is inconsistent. Line 544 uses spaces while line 545 uses tabs (or different spacing). The codebase should maintain consistent indentation. Based on the surrounding code context, use tabs for indentation to match the rest of the file.
| return false; | |
| return base.OnTouchEvent(e); | |
| return false; | |
| return base.OnTouchEvent(e); |
kubaflo
left a comment
There was a problem hiding this comment.
Can you rebase and fix public API please?
kubaflo
left a comment
There was a problem hiding this comment.
Could you please review the AI's summary?
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.sh | bash -s -- 27749Or
iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 27749" |
@kubaflo , I have addressed the AI review summary |
|
/review -b feature/refactor-copilot-yml |
MauiBot
left a comment
There was a problem hiding this comment.
🤖 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 passed the iOS regression test and fixes disabled CollectionView selection through native selection capability instead of UserInteractionEnabled, avoiding the PR fix's InputTransparent clobbering risk. try-fix-1 failed validation, and pr/pr-plus-reviewer are ranked lower because they rely on broader UIKit interaction state.
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/Handlers/Items/SelectableItemsViewHandler.cs b/src/Controls/src/Core/Handlers/Items/SelectableItemsViewHandler.cs
index b551f6fadd..ece5126c0b 100644
--- a/src/Controls/src/Core/Handlers/Items/SelectableItemsViewHandler.cs
+++ b/src/Controls/src/Core/Handlers/Items/SelectableItemsViewHandler.cs
@@ -23,6 +23,9 @@ namespace Microsoft.Maui.Controls.Handlers.Items
[SelectableItemsView.SelectedItemProperty.PropertyName] = MapSelectedItem,
[SelectableItemsView.SelectedItemsProperty.PropertyName] = MapSelectedItems,
[SelectableItemsView.SelectionModeProperty.PropertyName] = MapSelectionMode,
+#if IOS || MACCATALYST
+ [Controls.VisualElement.IsEnabledProperty.PropertyName] = MapIsEnabled,
+#endif
};
}
diff --git a/src/Controls/src/Core/Handlers/Items/SelectableItemsViewHandler.iOS.cs b/src/Controls/src/Core/Handlers/Items/SelectableItemsViewHandler.iOS.cs
index b26c336b25..bbe2f15cea 100644
--- a/src/Controls/src/Core/Handlers/Items/SelectableItemsViewHandler.iOS.cs
+++ b/src/Controls/src/Core/Handlers/Items/SelectableItemsViewHandler.iOS.cs
@@ -34,5 +34,10 @@ namespace Microsoft.Maui.Controls.Handlers.Items
{
(handler.Controller as SelectableItemsViewController<TItemsView>)?.UpdateSelectionMode();
}
+
+ internal static void MapIsEnabled(SelectableItemsViewHandler<TItemsView> handler, SelectableItemsView itemsView)
+ {
+ (handler.Controller as SelectableItemsViewController<TItemsView>)?.UpdateSelectionMode();
+ }
}
}
diff --git a/src/Controls/src/Core/Handlers/Items/iOS/SelectableItemsViewController.cs b/src/Controls/src/Core/Handlers/Items/iOS/SelectableItemsViewController.cs
index 1f00d0f482..e58c45f596 100644
--- a/src/Controls/src/Core/Handlers/Items/iOS/SelectableItemsViewController.cs
+++ b/src/Controls/src/Core/Handlers/Items/iOS/SelectableItemsViewController.cs
@@ -25,7 +25,7 @@ namespace Microsoft.Maui.Controls.Handlers.Items
// _Only_ called if the user initiates the selection change; will not be called for programmatic selection
public override void ItemSelected(UICollectionView collectionView, NSIndexPath indexPath)
{
- if (ItemsView?.ItemsSource is null)
+ if (ItemsView?.ItemsSource is null || !ItemsView.IsExplicitlyEnabled)
{
return;
}
@@ -35,7 +35,7 @@ namespace Microsoft.Maui.Controls.Handlers.Items
// _Only_ called if the user initiates the selection change; will not be called for programmatic selection
public override void ItemDeselected(UICollectionView collectionView, NSIndexPath indexPath)
{
- if (ItemsView?.ItemsSource is null)
+ if (ItemsView?.ItemsSource is null || !ItemsView.IsExplicitlyEnabled)
{
return;
}
@@ -119,6 +119,7 @@ namespace Microsoft.Maui.Controls.Handlers.Items
void FormsSelectItem(NSIndexPath indexPath)
{
var mode = ItemsView.SelectionMode;
+ var isEnabled = ItemsView.IsExplicitlyEnabled;
switch (mode)
{
@@ -185,6 +186,7 @@ namespace Microsoft.Maui.Controls.Handlers.Items
internal void UpdateSelectionMode()
{
var mode = ItemsView.SelectionMode;
+ var isEnabled = ItemsView.IsExplicitlyEnabled;
switch (mode)
{
@@ -194,13 +196,13 @@ namespace Microsoft.Maui.Controls.Handlers.Items
ClearsSelectionOnViewWillAppear = true;
break;
case SelectionMode.Single:
- CollectionView.AllowsSelection = true;
+ CollectionView.AllowsSelection = isEnabled;
CollectionView.AllowsMultipleSelection = false;
ClearsSelectionOnViewWillAppear = false;
break;
case SelectionMode.Multiple:
- CollectionView.AllowsSelection = true;
- CollectionView.AllowsMultipleSelection = true;
+ CollectionView.AllowsSelection = isEnabled;
+ CollectionView.AllowsMultipleSelection = isEnabled;
ClearsSelectionOnViewWillAppear = false;
break;
}
diff --git a/src/Controls/src/Core/Handlers/Items2/CollectionViewHandler2.iOS.cs b/src/Controls/src/Core/Handlers/Items2/CollectionViewHandler2.iOS.cs
index 7ed8b7eea8..6fe356e234 100644
--- a/src/Controls/src/Core/Handlers/Items2/CollectionViewHandler2.iOS.cs
+++ b/src/Controls/src/Core/Handlers/Items2/CollectionViewHandler2.iOS.cs
@@ -78,6 +78,7 @@ namespace Microsoft.Maui.Controls.Handlers.Items2
[SelectableItemsView.SelectedItemProperty.PropertyName] = MapSelectedItem,
[SelectableItemsView.SelectedItemsProperty.PropertyName] = MapSelectedItems,
[SelectableItemsView.SelectionModeProperty.PropertyName] = MapSelectionMode,
+ [Controls.VisualElement.IsEnabledProperty.PropertyName] = MapIsEnabled,
[StructuredItemsView.HeaderTemplateProperty.PropertyName] = MapHeaderTemplate,
[StructuredItemsView.FooterTemplateProperty.PropertyName] = MapFooterTemplate,
[StructuredItemsView.HeaderProperty.PropertyName] = MapHeaderTemplate,
@@ -154,6 +155,11 @@ namespace Microsoft.Maui.Controls.Handlers.Items2
(handler.Controller as SelectableItemsViewController2<ReorderableItemsView>)?.UpdateSelectionMode();
}
+ internal static void MapIsEnabled(CollectionViewHandler2 handler, SelectableItemsView itemsView)
+ {
+ (handler.Controller as SelectableItemsViewController2<ReorderableItemsView>)?.UpdateSelectionMode();
+ }
+
// Structured
protected override UICollectionViewLayout SelectLayout()
diff --git a/src/Controls/src/Core/Handlers/Items2/iOS/SelectableItemsViewController2.cs b/src/Controls/src/Core/Handlers/Items2/iOS/SelectableItemsViewController2.cs
index cc02ddde50..3382f84dc7 100644
--- a/src/Controls/src/Core/Handlers/Items2/iOS/SelectableItemsViewController2.cs
+++ b/src/Controls/src/Core/Handlers/Items2/iOS/SelectableItemsViewController2.cs
@@ -26,7 +26,7 @@ namespace Microsoft.Maui.Controls.Handlers.Items2
// _Only_ called if the user initiates the selection change; will not be called for programmatic selection
public override void ItemSelected(UICollectionView collectionView, NSIndexPath indexPath)
{
- if (ItemsView?.ItemsSource is null)
+ if (ItemsView?.ItemsSource is null || !ItemsView.IsExplicitlyEnabled)
{
return;
}
@@ -36,7 +36,7 @@ namespace Microsoft.Maui.Controls.Handlers.Items2
// _Only_ called if the user initiates the selection change; will not be called for programmatic selection
public override void ItemDeselected(UICollectionView collectionView, NSIndexPath indexPath)
{
- if (ItemsView?.ItemsSource is null)
+ if (ItemsView?.ItemsSource is null || !ItemsView.IsExplicitlyEnabled)
{
return;
}
@@ -120,6 +120,7 @@ namespace Microsoft.Maui.Controls.Handlers.Items2
void FormsSelectItem(NSIndexPath indexPath)
{
var mode = ItemsView.SelectionMode;
+ var isEnabled = ItemsView.IsExplicitlyEnabled;
switch (mode)
{
@@ -186,6 +187,7 @@ namespace Microsoft.Maui.Controls.Handlers.Items2
internal void UpdateSelectionMode()
{
var mode = ItemsView.SelectionMode;
+ var isEnabled = ItemsView.IsExplicitlyEnabled;
switch (mode)
{
@@ -195,13 +197,13 @@ namespace Microsoft.Maui.Controls.Handlers.Items2
ClearsSelectionOnViewWillAppear = true;
break;
case SelectionMode.Single:
- CollectionView.AllowsSelection = true;
+ CollectionView.AllowsSelection = isEnabled;
CollectionView.AllowsMultipleSelection = false;
ClearsSelectionOnViewWillAppear = false;
break;
case SelectionMode.Multiple:
- CollectionView.AllowsSelection = true;
- CollectionView.AllowsMultipleSelection = true;
+ CollectionView.AllowsSelection = isEnabled;
+ CollectionView.AllowsMultipleSelection = isEnabled;
ClearsSelectionOnViewWillAppear = false;
break;
}
diff --git a/src/Controls/tests/TestCases.HostApp/Issues/Issue20274.xaml b/src/Controls/tests/TestCases.HostApp/Issues/Issue20274.xaml
new file mode 100644
index 0000000000..e16b14c3a8
--- /dev/null
+++ b/src/Controls/tests/TestCases.HostApp/Issues/Issue20274.xaml
@@ -0,0 +1,27 @@
+<?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.Issue20274"
+ xmlns:local="clr-namespace:Maui.Controls.Sample.Issues">
+
+ <Grid RowDefinitions="auto,auto, *">
+ <Button Text="Change State" AutomationId="Button" Clicked="Button_Clicked"/>
+ <Label Grid.Row="1" AutomationId="Label" HeightRequest="50" VerticalTextAlignment="Center" x:Name="label"/>
+ <CollectionView
+ Grid.Row="2" IsEnabled="False"
+ SelectionMode="Single" x:Name="collectionView" SelectionChanged="collectionView_SelectionChanged"
+ ItemsSource="{Binding Items}">
+ <CollectionView.ItemTemplate>
+ <DataTemplate>
+ <Label Margin="10"
+ Padding="10"
+ AutomationId="ItemsLabel"
+ VerticalTextAlignment="Center"
+ HorizontalTextAlignment="Start"
+ Text="{Binding Name}" />
+ </DataTemplate>
+ </CollectionView.ItemTemplate>
+ </CollectionView>
+ </Grid>
+
+</ContentPage>
\ No newline at end of file
diff --git a/src/Controls/tests/TestCases.HostApp/Issues/Issue20274.xaml.cs b/src/Controls/tests/TestCases.HostApp/Issues/Issue20274.xaml.cs
new file mode 100644
index 0000000000..ea0fc460a4
--- /dev/null
+++ b/src/Controls/tests/TestCases.HostApp/Issues/Issue20274.xaml.cs
@@ -0,0 +1,61 @@
+using System.Collections.ObjectModel;
+using System.ComponentModel;
+
+namespace Maui.Controls.Sample.Issues
+{
+ [Issue(IssueTracker.Github, 20274, "IsEnabled=False on CollectionView not working", PlatformAffected.All)]
+ public partial class Issue20274 : ContentPage
+ {
+ public ObservableCollection<CollectionViewItem> Items { get; set; }
+
+ public Issue20274()
+ {
+ Items = new ObservableCollection<CollectionViewItem>
+ {
+ new CollectionViewItem { Name = "Item 1" },
+ new CollectionViewItem { Name = "Item 2" },
+ new CollectionViewItem { Name = "Item 3" }
+ };
+ InitializeComponent();
+ BindingContext = this;
+ label.Text = Items[0].Name;
+ }
+
+ void collectionView_SelectionChanged(object sender, SelectionChangedEventArgs e)
+ {
+ if (e.CurrentSelection?.FirstOrDefault() is CollectionViewItem selectedItem)
+ {
+ selectedItem.Name = $"{selectedItem.Name} - Selected";
+ this.label.Text = selectedItem.Name;
+ }
+ }
+
+ void Button_Clicked(object sender, EventArgs e)
+ {
+ collectionView.IsEnabled = !collectionView.IsEnabled;
+ }
+ }
+
+ public class CollectionViewItem : INotifyPropertyChanged
+ {
+ string _name;
+ public string Name
+ {
+ get => _name;
+ set
+ {
+ if (_name != value)
+ {
+ _name = value;
+ OnPropertyChanged(nameof(Name));
+ }
+ }
+ }
+
+ public event PropertyChangedEventHandler PropertyChanged;
+ protected void OnPropertyChanged(string propertyName)
+ {
+ PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
+ }
+ }
+}
\ No newline at end of file
diff --git a/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20274.cs b/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20274.cs
new file mode 100644
index 0000000000..bca296e894
--- /dev/null
+++ b/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20274.cs
@@ -0,0 +1,33 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using NUnit.Framework;
+using UITest.Appium;
+using UITest.Core;
+
+namespace Microsoft.Maui.TestCases.Tests.Issues
+{
+ public class Issue20274 : _IssuesUITest
+ {
+ public override string Issue => "IsEnabled=False on CollectionView not working";
+
+ public Issue20274(TestDevice device) : base(device)
+ {
+ }
+
+
+ [Test]
+ [Category(UITestCategories.CollectionView)]
+ public void VerifyCollectionViewIsEnableState()
+ {
+ App.WaitForElement("Button");
+ App.Tap("ItemsLabel");
+ Assert.That(App.FindElement("Label").GetText(), Is.EqualTo("Item 1"));
+ App.Tap("Button");
+ App.Tap("ItemsLabel");
+ Assert.That(App.FindElement("Label").GetText(), Is.EqualTo("Item 1 - Selected"));
+ }
+ }
+}
\ No newline at end of file
kubaflo
left a comment
There was a problem hiding this comment.
Could you please check the ai's suggestions?
@kubaflo , Based on AI suggestion, I have modified the fix |
|
/review -b feature/refactor-copilot-yml -p ios |
MauiBot
left a comment
There was a problem hiding this comment.
🤖 Automated review — alternative fix proposed
The expert-reviewer evaluation compared the PR fix against #1 automatically generated candidates and selected try-fix-1 as the strongest fix.
Why: try-fix-1 passed the requested iOS regression test and fixes the root native disabled-state behavior by propagating IsEnabled to UICollectionView.UserInteractionEnabled. It is simpler and safer than the raw PR and pr-plus-reviewer because it avoids unnecessary selection-controller changes while covering MacCatalyst mapper registration.
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-1`)
diff --git a/src/Controls/src/Core/Handlers/Items/ItemsViewHandler.cs b/src/Controls/src/Core/Handlers/Items/ItemsViewHandler.cs
index b10fa478b7..b08b129f9c 100644
--- a/src/Controls/src/Core/Handlers/Items/ItemsViewHandler.cs
+++ b/src/Controls/src/Core/Handlers/Items/ItemsViewHandler.cs
@@ -28,7 +28,10 @@ namespace Microsoft.Maui.Controls.Handlers.Items
[Controls.ItemsView.EmptyViewTemplateProperty.PropertyName] = MapEmptyViewTemplate,
[Controls.ItemsView.FlowDirectionProperty.PropertyName] = MapFlowDirection,
[Controls.ItemsView.IsVisibleProperty.PropertyName] = MapIsVisible,
- [Controls.ItemsView.ItemsUpdatingScrollModeProperty.PropertyName] = MapItemsUpdatingScrollMode
+ [Controls.ItemsView.ItemsUpdatingScrollModeProperty.PropertyName] = MapItemsUpdatingScrollMode,
+#if IOS || MACCATALYST
+ [nameof(IView.IsEnabled)] = MapIsEnabled,
+#endif
};
}
}
diff --git a/src/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cs b/src/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cs
index 846728b101..12754a2b56 100644
--- a/src/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cs
+++ b/src/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cs
@@ -91,6 +91,12 @@ namespace Microsoft.Maui.Controls.Handlers.Items
handler._layout.ItemsUpdatingScrollMode = itemsView.ItemsUpdatingScrollMode;
}
+ internal static void MapIsEnabled(ItemsViewHandler<TItemsView> handler, ItemsView itemsView)
+ {
+ ViewHandler.MapIsEnabled(handler, itemsView);
+ handler.Controller?.CollectionView?.UpdateIsEnabled(itemsView);
+ }
+
protected virtual void UpdateLayout()
{
_layout = SelectLayout();
diff --git a/src/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cs b/src/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cs
index 8f55ca5eb2..0db2d6bf1b 100644
--- a/src/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cs
+++ b/src/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cs
@@ -33,7 +33,8 @@ namespace Microsoft.Maui.Controls.Handlers.Items2
[Controls.ItemsView.EmptyViewTemplateProperty.PropertyName] = MapEmptyViewTemplate,
[Controls.ItemsView.FlowDirectionProperty.PropertyName] = MapFlowDirection,
[Controls.ItemsView.IsVisibleProperty.PropertyName] = MapIsVisible,
- [Controls.ItemsView.ItemsUpdatingScrollModeProperty.PropertyName] = MapItemsUpdatingScrollMode
+ [Controls.ItemsView.ItemsUpdatingScrollModeProperty.PropertyName] = MapItemsUpdatingScrollMode,
+ [nameof(IView.IsEnabled)] = MapIsEnabled
};
UICollectionViewLayout _layout;
@@ -121,6 +122,12 @@ namespace Microsoft.Maui.Controls.Handlers.Items2
handler.Controller?.UpdateVisibility();
}
+ internal static void MapIsEnabled(ItemsViewHandler2<TItemsView> handler, ItemsView itemsView)
+ {
+ ViewHandler.MapIsEnabled(handler, itemsView);
+ handler.Controller?.CollectionView?.UpdateIsEnabled(itemsView);
+ }
+
public static void MapItemsUpdatingScrollMode(ItemsViewHandler2<TItemsView> handler, ItemsView itemsView)
{
if (handler.ItemsView is StructuredItemsView structuredItemsView && structuredItemsView.ItemsLayout is ItemsLayout itemsLayout)
🤖 AI Summary
📊 Review Session —
|
| Test | Without Fix (expect FAIL) | With Fix (expect PASS) |
|---|---|---|
🖥️ Issue20274 Issue20274 |
✅ FAIL — 114s | ✅ PASS — 105s |
🔴 Without fix — 🖥️ Issue20274: FAIL ✅ · 114s
Determining projects to restore...
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 661 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 680 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 652 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 730 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 713 ms).
6 of 11 projects are up-to-date for restore.
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0-ios26.0/Microsoft.Maui.Graphics.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0-ios26.0/Microsoft.Maui.Essentials.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0-ios26.0/Microsoft.Maui.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Maps.dll
Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Controls.Xaml -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Xaml/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Xaml.dll
Microsoft.AspNetCore.Components.WebView.Maui -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Microsoft.AspNetCore.Components.WebView.Maui/Debug/net10.0-ios26.0/Microsoft.AspNetCore.Components.WebView.Maui.dll
Controls.Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Maps.dll
Controls.Foldable -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Foldable/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Foldable.dll
Detected signing identity:
Code Signing Key: "" (-)
Provisioning Profile: "" () - no entitlements
Bundle Id: com.microsoft.maui.uitests
App Id: com.microsoft.maui.uitests
Controls.TestCases.HostApp -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-ios/iossimulator-arm64/Controls.TestCases.HostApp.dll
Optimizing assemblies for size may change the behavior of the app. Be sure to test after publishing. See: https://aka.ms/dotnet-illink
Optimizing assemblies for size. This process might take a while.
Build succeeded.
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
1 Warning(s)
0 Error(s)
Time Elapsed 00:00:56.83
Determining projects to restore...
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 363 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 370 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 369 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 393 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 409 ms).
8 of 13 projects are up-to-date for restore.
Controls.CustomAttributes -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.CustomAttributes/Debug/net10.0/Controls.CustomAttributes.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0/Microsoft.Maui.Graphics.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0/Microsoft.Maui.dll
Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0/Microsoft.Maui.Controls.dll
VisualTestUtils -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils/Debug/netstandard2.0/VisualTestUtils.dll
VisualTestUtils.MagickNet -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils.MagickNet/Debug/netstandard2.0/VisualTestUtils.MagickNet.dll
UITest.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Core/Debug/net10.0/UITest.Core.dll
UITest.Appium -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.dll
UITest.NUnit -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.NUnit/Debug/net10.0/UITest.NUnit.dll
UITest.Analyzers -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Analyzers/Debug/netstandard2.0/UITest.Analyzers.dll
Controls.TestCases.iOS.Tests -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
Test run for /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (arm64)
Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
[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.06] Discovering: Controls.TestCases.iOS.Tests
[xUnit.net 00:00:00.17] Discovered: Controls.TestCases.iOS.Tests
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
NUnit3TestExecutor discovered 1 of 1 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 5/26/2026 3:16:59 PM FixtureSetup for Issue20274(iOS)
>>>>> 5/26/2026 3:17:03 PM VerifyCollectionViewIsEnableState Start
>>>>> 5/26/2026 3:17:04 PM VerifyCollectionViewIsEnableState Stop
>>>>> 5/26/2026 3:17:04 PM Log types: syslog, crashlog, performance, safariConsole, safariNetwork, server
Failed VerifyCollectionViewIsEnableState [987 ms]
Error Message:
Assert.That(App.FindElement("Label").GetText(), Is.EqualTo("Item 1"))
Expected string length 6 but was 17. Strings differ at index 6.
Expected: "Item 1"
But was: "Item 1 - Selected"
-----------------^
Stack Trace:
at Microsoft.Maui.TestCases.Tests.Issues.Issue20274.VerifyCollectionViewIsEnableState() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20274.cs:line 27
1) at Microsoft.Maui.TestCases.Tests.Issues.Issue20274.VerifyCollectionViewIsEnableState() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20274.cs:line 27
NUnit Adapter 4.5.0.0: Test execution complete
Results File: /Users/cloudtest/vss/_work/1/s/CustomAgentLogsTmp/UITests/TestResults/Issue20274.trx
Test Run Failed.
Total tests: 1
Failed: 1
Total time: 22.0509 Seconds
>>> TRX_RESULT_FILE: /Users/cloudtest/vss/_work/1/s/CustomAgentLogsTmp/UITests/TestResults/Issue20274.trx
🟢 With fix — 🖥️ Issue20274: PASS ✅ · 105s
Determining projects to restore...
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 389 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 361 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 409 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 453 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 471 ms).
6 of 11 projects are up-to-date for restore.
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0-ios26.0/Microsoft.Maui.Graphics.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0-ios26.0/Microsoft.Maui.Essentials.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0-ios26.0/Microsoft.Maui.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Maps.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Microsoft.AspNetCore.Components.WebView.Maui -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Microsoft.AspNetCore.Components.WebView.Maui/Debug/net10.0-ios26.0/Microsoft.AspNetCore.Components.WebView.Maui.dll
Controls.Foldable -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Foldable/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Foldable.dll
Controls.Xaml -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Xaml/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Xaml.dll
Controls.Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Maps.dll
Detected signing identity:
Code Signing Key: "" (-)
Provisioning Profile: "" () - no entitlements
Bundle Id: com.microsoft.maui.uitests
App Id: com.microsoft.maui.uitests
Controls.TestCases.HostApp -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-ios/iossimulator-arm64/Controls.TestCases.HostApp.dll
Optimizing assemblies for size may change the behavior of the app. Be sure to test after publishing. See: https://aka.ms/dotnet-illink
Optimizing assemblies for size. This process might take a while.
Build succeeded.
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
1 Warning(s)
0 Error(s)
Time Elapsed 00:00:52.39
Determining projects to restore...
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 393 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 413 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 413 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 436 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 453 ms).
8 of 13 projects are up-to-date for restore.
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Controls.CustomAttributes -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.CustomAttributes/Debug/net10.0/Controls.CustomAttributes.dll
Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0/Microsoft.Maui.Graphics.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0/Microsoft.Maui.dll
Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
##vso[build.updatebuildnumber]10.0.80-ci+azdo.14198706
Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0/Microsoft.Maui.Controls.dll
UITest.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Core/Debug/net10.0/UITest.Core.dll
VisualTestUtils -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils/Debug/netstandard2.0/VisualTestUtils.dll
UITest.NUnit -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.NUnit/Debug/net10.0/UITest.NUnit.dll
VisualTestUtils.MagickNet -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils.MagickNet/Debug/netstandard2.0/VisualTestUtils.MagickNet.dll
UITest.Appium -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.dll
UITest.Analyzers -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Analyzers/Debug/netstandard2.0/UITest.Analyzers.dll
Controls.TestCases.iOS.Tests -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
Test run for /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (arm64)
Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
[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.05] Discovering: Controls.TestCases.iOS.Tests
[xUnit.net 00:00:00.18] Discovered: Controls.TestCases.iOS.Tests
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
NUnit3TestExecutor discovered 1 of 1 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 5/26/2026 3:18:44 PM FixtureSetup for Issue20274(iOS)
>>>>> 5/26/2026 3:18:47 PM VerifyCollectionViewIsEnableState Start
>>>>> 5/26/2026 3:18:49 PM VerifyCollectionViewIsEnableState Stop
Passed VerifyCollectionViewIsEnableState [1 s]
NUnit Adapter 4.5.0.0: Test execution complete
Results File: /Users/cloudtest/vss/_work/1/s/CustomAgentLogsTmp/UITests/TestResults/Issue20274.trx
Test Run Successful.
Total tests: 1
Passed: 1
Total time: 22.4895 Seconds
>>> TRX_RESULT_FILE: /Users/cloudtest/vss/_work/1/s/CustomAgentLogsTmp/UITests/TestResults/Issue20274.trx
📁 Fix files reverted (6 files)
eng/pipelines/ci-copilot.ymlsrc/Controls/src/Core/Handlers/Items/ItemsViewHandler.cssrc/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cssrc/Controls/src/Core/Handlers/Items/iOS/SelectableItemsViewController.cssrc/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cssrc/Controls/src/Core/Handlers/Items2/iOS/SelectableItemsViewController2.cs
🧪 UI Tests — CollectionView
Detected UI test categories: CollectionView
✅ Deep UI tests — 416 passed, 0 failed across 1 category on platform-pool agent (replaces in-process counts above).
🧪 UI Test Execution Results (deep, platform pool)
| Category | Tests | Snapshot diffs |
|---|---|---|
CollectionView |
416/418 ✓ | — |
📎 Download drop-deep-uitests artifact (TRX + snapshot diffs) |
🔍 Regression Cross-Reference
🔍 Regression Cross-Reference
🟢 No regression risks detected. No labeled bug-fix PRs in the last 6 months touched the modified files.
🔍 Pre-Flight — Context & Validation
Issue: #27770 - [iOS/Android] CollectionView IsEnabled Not Working
PR: #27749 - [Android/iOS] Fix IsEnabled=False on CollectionView not working
Platforms Affected: iOS, Android (testing platform requested: ios)
Files Changed: 5 implementation, 3 test
Key Findings
- The issue reports that
CollectionView.IsEnabled=falsestill permits user interaction/selection on iOS and Android. - The current PR diff in the review branch changes iOS legacy
Items/and currentItems2/handlers plus an Appium UI regression test; the PR narrative still mentions Android changes, but the available diff is Apple handler/test focused. - Gate was already completed externally: tests fail without the fix and pass with the PR fix. This phase did not re-run gate verification and did not create or overwrite
gate/content.md. - Review comments raised cross-platform behavior and
InputTransparentconsiderations; current active platform for this run is iOS. - The new UI test is a
CollectionViewAppium test namedIssue20274despite the linked issue being [iOS/Android] CollectionView IsEnabled Not Working #27770.
Code Review Summary
Verdict: NEEDS_CHANGES
Confidence: high
Errors: 3 | Warnings: 0 | Suggestions: 0
Key code review findings:
- ❌
src/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cs:102- the newIsEnabledmapper replaces inherited native enabled-state mapping but only updates selection mode. - ❌
src/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cs:127- same CV2 issue: runtimeIsEnabledchanges do not preserve nativeUICollectionView.UserInteractionEnabledupdates. - ❌
src/Controls/src/Core/Handlers/Items/ItemsViewHandler.cs:32- legacy mapper entry is behind#if IOS; if legacy Apple behavior is intended for MacCatalyst too, it should includeMACCATALYST.
Fix Candidates
| # | Source | Approach | Test Result | Files Changed | Notes |
|---|---|---|---|---|---|
| PR | PR #27749 | Add iOS mapper entries, block selection callbacks when !IsExplicitlyEnabled, and gate AllowsSelection / AllowsMultipleSelection by enabled state. |
✅ PASSED (Gate) | 5 implementation, 3 test | Original PR; gate result supplied by caller. |
🔬 Code Review — Deep Analysis
Code Review — PR #27749
Independent Assessment
What this changes: Adds iOS/CV2 CollectionView IsEnabled mapping so selection callbacks and AllowsSelection/AllowsMultipleSelection respect IsExplicitlyEnabled; adds an Appium UI test page for disabled CollectionView selection.
Inferred motivation: Prevent user item selection when CollectionView.IsEnabled=false.
Reconciliation with PR Narrative
Author claims: Fixes CollectionView IsEnabled=false for Android/iOS, disables native interaction, tested Android/Windows/iOS/Mac.
Agreement/disagreement: Current diff is Apple handler/test-only. More importantly, the new mapper blocks selection but does not preserve native UserInteractionEnabled updates.
Findings
❌ Error — Native enabled state no longer updates in legacy iOS handler
src/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cs:102
The new IsEnabled mapper replaces inherited ViewHandler.MapIsEnabled, but only calls UpdateSelectionMode(). Base behavior calls UIView.UpdateIsEnabled, which sets UserInteractionEnabled. Disabled CollectionViews can still scroll/receive non-selection gestures. Preserve native enabled mapping before updating selection mode.
❌ Error — Native enabled state no longer updates in CV2 handler
src/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cs:127
Same issue for CV2: the override replaces base mapping and only toggles selection. UICollectionView.UserInteractionEnabled is not kept in sync for runtime IsEnabled changes.
❌ Error — MacCatalyst missing legacy mapper entry
src/Controls/src/Core/Handlers/Items/ItemsViewHandler.cs:32
The mapper entry is behind #if IOS, but the paired .iOS.cs implementation also compiles for MacCatalyst. Use #if IOS || MACCATALYST if this behavior is intended for both Apple platforms.
Devil's Advocate
The selection guards do address the specific tap-selection path, but the PR narrative claims native interaction is disabled; replacing the base mapper proves that is not true for other interactions. The MacCatalyst concern follows existing repo patterns (IOS || MACCATALYST) and platform-file rules.
Verdict: NEEDS_CHANGES
Confidence: high
Summary: The core fix is incomplete because it regresses native IsEnabled handling while adding selection-only behavior. CI is also currently failing (maui-pr/Build Analysis), so this is not ready to merge. Inline findings were written to CustomAgentLogsTmp/PRState/27749/PRAgent/inline-findings.json.
🔧 Fix — Analysis & Comparison
Fix Candidates
| # | Source | Approach | Test Result | Files Changed | Notes |
|---|---|---|---|---|---|
| 1 | try-fix-1 | Native iOS interaction gating via ViewHandler.MapIsEnabled + UICollectionView.UpdateIsEnabled; leaves selection callbacks/mode unchanged. |
✅ PASS | 3 files | Better than PR fix: preserves native enabled behavior, handles scrolling/non-selection gestures, includes MacCatalyst mapper guard, and avoids controller selection logic changes. |
| PR | PR #27749 | Selection-callback guards plus enabled-state gating of AllowsSelection / AllowsMultipleSelection. |
✅ PASSED (Gate) | 5 implementation, 3 test | Original PR; gate result supplied by caller. Code review found native enabled-state preservation concerns. |
Cross-Pollination
| Model | Round | New Ideas? | Details |
|---|---|---|---|
| maui-expert-reviewer | 1 | Yes | Proposed native interaction gating with ViewHandler.MapIsEnabled and UICollectionView.UpdateIsEnabled, not selection delegate guards. |
Exhausted: No — stopped because candidate #1 passed all requested iOS regression testing and is demonstrably better than the PR fix.
Selected Fix: Candidate #1 — It passes the iOS UI regression test while preserving native IsEnabled semantics and reducing the implementation surface from selection controller logic to handler-level native enabled-state propagation.
try-fix-1 — Native Interaction Gating
Approach Description
Use native iOS interaction disabling instead of selection-callback guards. The candidate adds IsEnabled mapper entries for Apple CollectionView handlers, calls ViewHandler.MapIsEnabled, and explicitly applies UpdateIsEnabled to the underlying UICollectionView.
Test Results
- Command:
pwsh .github/scripts/BuildAndRunHostApp.ps1 -Platform ios -TestFilter "Issue20274" - Result: ✅ Pass
- Evidence:
attempt-1/test-output.logreports✅ All tests passedandResult: SUCCESS ✅.
Failure Analysis
Not applicable; candidate passed.
Diff
See CustomAgentLogsTmp/PRState/27749/PRAgent/try-fix-1/content.md and CustomAgentLogsTmp/PRState/27749/PRAgent/try-fix/attempt-1/fix.diff.
📋 Report — Final Recommendation
Comparative Report — PR #27749
Candidates compared
| Rank | Candidate | Regression result | Assessment |
|---|---|---|---|
| 1 | try-fix-1 |
✅ PASS | Best candidate. It fixes the root iOS behavior by propagating IsEnabled to the native UICollectionView.UserInteractionEnabled state, includes the MacCatalyst mapper guard, and leaves selection callbacks/mode semantics unchanged. |
| 2 | pr-plus-reviewer |
Not re-run in this phase | Addresses the expert review findings by adding native enabled-state propagation and MacCatalyst coverage to the PR fix. It should resolve the raw PR's correctness issues, but it retains extra selection-controller changes that are unnecessary once native interaction is disabled. |
| 3 | pr |
✅ PASS (supplied gate) | Passes the targeted selection regression, but expert review found major correctness issues: the custom IsEnabled mapper replaces inherited native enabled-state mapping, so disabled CollectionViews can still scroll/receive non-selection native interaction; legacy MacCatalyst coverage is also missing. |
Analysis
The raw PR fix is correctly aimed at preventing disabled CollectionViews from changing selection, and the supplied gate confirms the new iOS regression test fails without the fix and passes with the PR. However, the implementation fixes the symptom at the selection delegate/selection-mode layer rather than the native enabled-state layer. Because UICollectionView is not a UIControl, correct disabled behavior requires updating UserInteractionEnabled; replacing the inherited IView.IsEnabled mapper without preserving that behavior leaves non-selection interaction enabled.
pr-plus-reviewer is a viable PR-derived candidate because it incorporates the expert feedback: it restores native enabled-state propagation and includes MacCatalyst in the legacy handler mapper guard. That removes the major review findings while keeping the PR's tests. Its downside is unnecessary complexity: the additional ItemSelected / ItemDeselected guards and AllowsSelection gating duplicate behavior that native interaction disabling already provides.
try-fix-1 is the cleanest and most robust solution. It directly updates the native interaction surface through the handler mapper, uses the existing UICollectionView.UpdateIsEnabled path, covers both iOS and MacCatalyst mapper registration where applicable, and avoids touching selection controller logic. It also passed the requested iOS regression command in STEP 6a.
Winner
try-fix-1 wins. It passed regression testing and fixes the broader disabled interaction behavior with a smaller, more idiomatic MAUI handler change than either the raw PR or the PR-plus-reviewer candidate.
📊 Review Session — 60669f6 · android fix removed · 2026-05-12 18:53 UTC
🚦 Gate — Test Before & After Fix
Gate Result: ✅ PASSED
Platform: IOS · Base: main · Merge base: f8cb875e
| Test | Without Fix (expect FAIL) | With Fix (expect PASS) |
|---|---|---|
🖥️ Issue20274 Issue20274 |
✅ FAIL — 238s | ✅ PASS — 97s |
🔴 Without fix — 🖥️ Issue20274: FAIL ✅ · 238s
Determining projects to restore...
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 533 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 690 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 5.37 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/Foldable/src/Controls.Foldable.csproj (in 5.56 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Xaml/Controls.Xaml.csproj (in 5.57 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/Core/maps/src/Maps.csproj (in 5.57 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 5.57 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 5.58 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/BlazorWebView/src/Maui/Microsoft.AspNetCore.Components.WebView.Maui.csproj (in 5.57 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj (in 5.57 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/Maps/src/Controls.Maps.csproj (in 5.58 sec).
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0-ios26.0/Microsoft.Maui.Graphics.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0-ios26.0/Microsoft.Maui.Essentials.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0-ios26.0/Microsoft.Maui.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Maps.dll
Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Microsoft.AspNetCore.Components.WebView.Maui -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Microsoft.AspNetCore.Components.WebView.Maui/Debug/net10.0-ios26.0/Microsoft.AspNetCore.Components.WebView.Maui.dll
Controls.Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Maps.dll
Controls.Foldable -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Foldable/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Foldable.dll
Controls.Xaml -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Xaml/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Xaml.dll
Detected signing identity:
Code Signing Key: "" (-)
Provisioning Profile: "" () - no entitlements
Bundle Id: com.microsoft.maui.uitests
App Id: com.microsoft.maui.uitests
Controls.TestCases.HostApp -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-ios/iossimulator-arm64/Controls.TestCases.HostApp.dll
Optimizing assemblies for size may change the behavior of the app. Be sure to test after publishing. See: https://aka.ms/dotnet-illink
Optimizing assemblies for size. This process might take a while.
Build succeeded.
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
1 Warning(s)
0 Error(s)
Time Elapsed 00:02:01.38
Determining projects to restore...
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 740 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.Core/UITest.Core.csproj (in 739 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/tests/CustomAttributes/Controls.CustomAttributes.csproj (in 740 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/VisualTestUtils/VisualTestUtils.csproj (in 740 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 745 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 795 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 868 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 882 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.NUnit/UITest.NUnit.csproj (in 1.48 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.Appium/UITest.Appium.csproj (in 1.77 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.Analyzers/UITest.Analyzers.csproj (in 1.97 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/VisualTestUtils.MagickNet/VisualTestUtils.MagickNet.csproj (in 3.86 sec).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.iOS.Tests/Controls.TestCases.iOS.Tests.csproj (in 4.65 sec).
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0/Microsoft.Maui.Graphics.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
Controls.CustomAttributes -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.CustomAttributes/Debug/net10.0/Controls.CustomAttributes.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0/Microsoft.Maui.dll
Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0/Microsoft.Maui.Controls.dll
UITest.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Core/Debug/net10.0/UITest.Core.dll
VisualTestUtils -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils/Debug/netstandard2.0/VisualTestUtils.dll
UITest.NUnit -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.NUnit/Debug/net10.0/UITest.NUnit.dll
VisualTestUtils.MagickNet -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils.MagickNet/Debug/netstandard2.0/VisualTestUtils.MagickNet.dll
UITest.Appium -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.dll
UITest.Analyzers -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Analyzers/Debug/netstandard2.0/UITest.Analyzers.dll
Controls.TestCases.iOS.Tests -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
Test run for /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (arm64)
Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
/Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.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.05] Discovering: Controls.TestCases.iOS.Tests
[xUnit.net 00:00:00.16] Discovered: Controls.TestCases.iOS.Tests
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
NUnit3TestExecutor discovered 1 of 1 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 5/12/2026 7:58:40 AM FixtureSetup for Issue20274(iOS)
>>>>> 5/12/2026 7:58:44 AM VerifyCollectionViewIsEnableState Start
>>>>> 5/12/2026 7:58:44 AM VerifyCollectionViewIsEnableState Stop
>>>>> 5/12/2026 7:58:44 AM Log types: syslog, crashlog, performance, safariConsole, safariNetwork, server
Failed VerifyCollectionViewIsEnableState [907 ms]
Error Message:
Assert.That(App.FindElement("Label").GetText(), Is.EqualTo("Item 1"))
Expected string length 6 but was 17. Strings differ at index 6.
Expected: "Item 1"
But was: "Item 1 - Selected"
-----------------^
Stack Trace:
at Microsoft.Maui.TestCases.Tests.Issues.Issue20274.VerifyCollectionViewIsEnableState() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20274.cs:line 27
1) at Microsoft.Maui.TestCases.Tests.Issues.Issue20274.VerifyCollectionViewIsEnableState() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20274.cs:line 27
NUnit Adapter 4.5.0.0: Test execution complete
Total tests: 1
Failed: 1
Test Run Failed.
Total time: 1.2249 Minutes
🟢 With fix — 🖥️ Issue20274: PASS ✅ · 97s
Determining projects to restore...
Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 408 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 406 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 401 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 444 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 457 ms).
6 of 11 projects are up-to-date for restore.
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0-ios26.0/Microsoft.Maui.Graphics.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0-ios26.0/Microsoft.Maui.Essentials.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0-ios26.0/Microsoft.Maui.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Maps.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Microsoft.AspNetCore.Components.WebView.Maui -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Microsoft.AspNetCore.Components.WebView.Maui/Debug/net10.0-ios26.0/Microsoft.AspNetCore.Components.WebView.Maui.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Controls.Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Maps.dll
Controls.Foldable -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Foldable/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Foldable.dll
Controls.Xaml -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Xaml/Debug/net10.0-ios26.0/Microsoft.Maui.Controls.Xaml.dll
Detected signing identity:
Code Signing Key: "" (-)
Provisioning Profile: "" () - no entitlements
Bundle Id: com.microsoft.maui.uitests
App Id: com.microsoft.maui.uitests
Controls.TestCases.HostApp -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.HostApp/Debug/net10.0-ios/iossimulator-arm64/Controls.TestCases.HostApp.dll
Optimizing assemblies for size may change the behavior of the app. Be sure to test after publishing. See: https://aka.ms/dotnet-illink
Optimizing assemblies for size. This process might take a while.
Build succeeded.
/Users/cloudtest/vss/_work/1/s/.dotnet/packs/Microsoft.iOS.Sdk.net10.0_26.0/26.0.11017/targets/Xamarin.Shared.Sdk.targets(309,3): warning : RuntimeIdentifier was set on the command line, and will override the value for RuntimeIdentifiers set in the project file. [/Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj::TargetFramework=net10.0-ios]
1 Warning(s)
0 Error(s)
Time Elapsed 00:00:48.15
Determining projects to restore...
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 352 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 400 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 399 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 425 ms).
Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 445 ms).
8 of 13 projects are up-to-date for restore.
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Controls.CustomAttributes -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.CustomAttributes/Debug/net10.0/Controls.CustomAttributes.dll
Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0/Microsoft.Maui.Graphics.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0/Microsoft.Maui.dll
Controls.BindingSourceGen -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.BindingSourceGen/Debug/netstandard2.0/Microsoft.Maui.Controls.BindingSourceGen.dll
##vso[build.updatebuildnumber]10.0.70-ci+azdo.14075589
Controls.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.Core/Debug/net10.0/Microsoft.Maui.Controls.dll
UITest.Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Core/Debug/net10.0/UITest.Core.dll
VisualTestUtils -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils/Debug/netstandard2.0/VisualTestUtils.dll
VisualTestUtils.MagickNet -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/VisualTestUtils.MagickNet/Debug/netstandard2.0/VisualTestUtils.MagickNet.dll
UITest.NUnit -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.NUnit/Debug/net10.0/UITest.NUnit.dll
UITest.Appium -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.dll
UITest.Analyzers -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Analyzers/Debug/netstandard2.0/UITest.Analyzers.dll
Controls.TestCases.iOS.Tests -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
Test run for /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (arm64)
Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
/Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.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.04] Discovering: Controls.TestCases.iOS.Tests
[xUnit.net 00:00:00.14] Discovered: Controls.TestCases.iOS.Tests
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in /Users/cloudtest/vss/_work/1/s/artifacts/bin/Controls.TestCases.iOS.Tests/Debug/net10.0/Controls.TestCases.iOS.Tests.dll
NUnit3TestExecutor discovered 1 of 1 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 5/12/2026 8:00:17 AM FixtureSetup for Issue20274(iOS)
>>>>> 5/12/2026 8:00:20 AM VerifyCollectionViewIsEnableState Start
>>>>> 5/12/2026 8:00:22 AM VerifyCollectionViewIsEnableState Stop
Passed VerifyCollectionViewIsEnableState [1 s]
NUnit Adapter 4.5.0.0: Test execution complete
Test Run Successful.
Total tests: 1
Passed: 1
Total time: 21.7715 Seconds
📁 Fix files reverted (3 files)
src/Controls/src/Core/Handlers/Items/ItemsViewHandler.cssrc/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cssrc/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cs
🧪 UI Tests — CollectionView
Detected UI test categories: CollectionView
✅ Deep UI tests — 416 passed, 0 failed across 1 category on platform-pool agent (replaces in-process counts above).
🧪 UI Test Execution Results (deep, platform pool)
| Category | Tests | Snapshot diffs |
|---|---|---|
CollectionView |
416/418 ✓ | — |
📎 Download drop-deep-uitests artifact (TRX + snapshot diffs) |
🔍 Pre-Flight — Context & Validation
Issue: #27770 — [iOS/Android] CollectionView IsEnabled Not Working (closely related to #20274, which is the issue number used in the test files)
PR: #27749 — [Android/iOS] Fix IsEnabled=False on CollectionView not working
Platforms Affected: iOS (and MacCatalyst — but not handled by this PR). PR title and description also claim Android, but no Android code is in the diff.
Files Changed: 3 implementation (all iOS CollectionView handlers), 3 test (one UITest + xaml + xaml.cs HostApp page).
Key Findings
- The PR registers a new
IsEnabledmapper onItemsViewHandler(legacy CV) andItemsViewHandler2(CV2) that setsController.CollectionView.UserInteractionEnabled = itemsView.IsEnabled. This works because the legacy handler'sPlatformViewisController.View(an outer wrapper), so the baseViewHandler.MapIsEnabledonly disables the wrapper — touches still reach the innerUICollectionView. - The legacy mapper registration is gated by
#if IOS, which excludes MacCatalyst (theIOSsymbol is iOS-only; MacCatalyst usesMACCATALYST). This contradicts the PR description, which claims Mac was tested. - The
ItemsViewHandler2registration lives inItemsViewHandler2.iOS.csand is not guarded — so CV2 picks up the fix on both iOS and MacCatalyst, but legacy CV does not. Inconsistent platform coverage between the two handlers. - PR description claims "Android: Disabled PlatformView and restricted touch interactions by overriding OnTouchEvent in MauiRecyclerView," but the actual diff contains zero Android changes (the prior
MauiRecyclerView.csedit visible in old inline comments was removed during rebase). - Both new mappers fully shadow the base
ViewHandler.MapIsEnabled(same keynameof(IView.IsEnabled)). The base'sUpdateIsEnabledno longer runs for these handlers; this happens to be benign on iOS but is a maintainability concern. - Test/HostApp uses issue number
20274in file names while the PRFixes #27770. Copilot already flagged this and a mutable model state issue (selection appends " - Selected" to boundName, breaking the test on re-runs).
Code Review Summary
Verdict: NEEDS_CHANGES
Confidence: high
Errors: 1 | Warnings: 4 | Suggestions: 4
Key code review findings:
- ❌
src/Controls/src/Core/Handlers/Items/ItemsViewHandler.cs:32—#if IOSexcludes MacCatalyst; legacy CV never gets the fix on Mac. Should be#if IOS || MACCATALYST, or better, move the mapper entry intoItemsViewHandler.iOS.cs(the pattern Items2 already uses). ⚠️ src/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cs:100— BaseViewHandler.MapIsEnabledis shadowed; considerAppendToMappingor call base first.⚠️ src/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cs:36— Same base-mapper shadowing.⚠️ PR description claims Android changes, but the diff has none — title and description are misleading.⚠️ src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20274.cs:23— Test will likely fail on MacCatalyst (legacy CV mapper not registered there) and is brittle becauseselectedItem.Nameis mutated on every selection.- 💡 Architectural inconsistency between Items legacy (#if-guarded in shared file) and Items2 (in .iOS.cs partial).
- 💡 Test coverage gap: doesn't verify toggling IsEnabled back to false.
- 💡 Minor indentation:
#if/#endifindented with tabs; house style is column 0. - 💡 HostApp page mutates bound
Name(appends " - Selected") — fragile on repeated taps.
Fix Candidates
| # | Source | Approach | Test Result | Files Changed | Notes |
|---|---|---|---|---|---|
| PR | PR #27749 | Register a separate IsEnabled mapper on ItemsViewHandler/ItemsViewHandler2 that sets Controller.CollectionView.UserInteractionEnabled on iOS only |
✅ PASSED (Gate, iOS) | 3 prod + 3 test | Original PR. MacCatalyst not covered for legacy CV; description claims Android changes that aren't present. |
🔬 Code Review — Deep Analysis
Code Review — PR #27749
Independent Assessment
What this changes: Adds an IsEnabled mapper to both ItemsViewHandler<TItemsView> (legacy CV) and ItemsViewHandler2<TItemsView> (CV2) that, on iOS, sets Controller.CollectionView.UserInteractionEnabled to match ItemsView.IsEnabled. Also adds a UI test (Issue20274) and HostApp page (xaml + code-behind) that toggle IsEnabled and verify selection is blocked/allowed accordingly.
Inferred motivation: The PlatformView returned by ItemsViewHandler.CreatePlatformView() is Controller.View — the outer UICollectionViewController.View wrapper — not the inner UICollectionView. The base ViewHandler.MapIsEnabled calls platformView.UpdateIsEnabled(view) on that outer wrapper, which (apparently) doesn't propagate to block selection on the inner UICollectionView reliably. So the fix explicitly disables interaction on the inner UICollectionView.
Reconciliation with PR Narrative
Author claims:
- "iOS: Disabled UserInteractionEnabled based on the IsEnabled state to prevent any user interaction." ✅ matches code.
- "Android: Disabled PlatformView and restricted touch interactions by overriding OnTouchEvent in MauiRecyclerView." ❌ no Android code in the actual diff (
gh pr view 27749 --json filesreturns 6 files, all iOS/test). Inline comment history shows there was once an Android edit atMauiRecyclerView.cs:545, but it's been removed. - "Tested on Android, Windows, iOS, Mac."
⚠️ The legacyItemsViewHandlermapper is gated by#if IOS, so MacCatalyst doesn't get the fix for the legacy CV path. CV2 is unguarded (lives in.iOS.cspartial) and does work on Mac, but legacy CV does not.
Agreement/disagreement: Agree the iOS fix is the right shape. Disagree that Mac is covered — there's an inconsistency between legacy CV (iOS only) and CV2 (iOS + Mac). Disagree that Android is fixed — no Android code is present.
Findings
❌ Error — MacCatalyst is excluded for legacy CV
src/Controls/src/Core/Handlers/Items/ItemsViewHandler.cs:32
#if IOS
[nameof(IView.IsEnabled)] = MapIsEnabled,
#endifThe IOS compilation symbol is defined only for net*-ios. MacCatalyst uses the MACCATALYST symbol. The reference platform code itself uses #if IOS || MACCATALYST for this kind of case (e.g., src/Core/src/Handlers/View/ViewHandler.cs:611). As written, the legacy ItemsViewHandler never registers the IsEnabled mapper on MacCatalyst, so the bug persists there.
Suggested fix: Either change to #if IOS || MACCATALYST, or move the entry into ItemsViewHandler.iOS.cs (the Items2 pattern already does this and works for both TFMs).
⚠️ Warning — Base ViewHandler mapper is shadowed (legacy)
src/Controls/src/Core/Handlers/Items/ItemsViewHandler.iOS.cs:100
The new mapper uses the same key (nameof(IView.IsEnabled)) as ViewHandler's base mapper, fully replacing it. The base mapper calls platformView.UpdateIsEnabled(view) on Controller.View — that's now skipped. On iOS this is currently benign because UpdateIsEnabled on the outer wrapper had no useful effect anyway, but future contributors who modify the base mapper (e.g., to add accessibility hooks) will silently miss CollectionView.
Suggested fix: Either invoke ViewHandler.MapIsEnabled(handler, itemsView) first, then set CollectionView.UserInteractionEnabled; or use Mapper.AppendToMapping(...) instead of overriding the dictionary slot.
⚠️ Warning — Base ViewHandler mapper is shadowed (Items2)
src/Controls/src/Core/Handlers/Items2/ItemsViewHandler2.iOS.cs:36
Same shadowing as above for CV2. Same suggested fix.
⚠️ Warning — PR description does not match the diff (Android missing)
The PR title says [Android/iOS] and the description includes an entire Android change paragraph and a "Tested on Android" checkbox, but the diff has no Android files. Either re-add the Android fix or update the title/description/before-after media to reflect iOS-only. The linked issue (#27770) is marked Android, iOS as affected platforms.
⚠️ Warning — Test brittleness and platform coverage
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20274.cs:23
The HostApp code (Issue20274.xaml.cs:29) mutates the bound model's Name on every selection by appending " - Selected". The test re-taps the same item after re-enabling — that works once, but if the test or any re-run order changes, the second tap would yield "Item 1 - Selected - Selected". Also, the test has [Category(UITestCategories.CollectionView)] with no platform exclusion, so on MacCatalyst (where legacy CV is not patched) the first assertion Is.EqualTo("Item 1") would fail.
Suggested fix: Set label.Text directly to $"{selectedItem.Name} - Selected" without mutating selectedItem.Name. Optionally add a [Platform] filter or wait until the MacCatalyst gap is closed.
💡 Suggestion — Architectural inconsistency between Items and Items2 registration sites
ItemsViewHandler.cs puts the entry in the shared file behind #if IOS; ItemsViewHandler2.iOS.cs puts it in the iOS partial. Pick one pattern. Putting both in the .iOS.cs partial is cleaner (no preprocessor in shared file) and naturally covers MacCatalyst since .iOS.cs compiles for both TFMs.
💡 Suggestion — Add re-toggle coverage
The test toggles IsEnabled once (false → true). Consider asserting that toggling back to false again re-blocks selection.
💡 Suggestion — Style: preprocessor indentation
#if IOS / #endif should be at column 0 per repo style.
💡 Suggestion — Don't mutate bound model state
Move the " - Selected" suffix into the Label.Text assignment rather than the model property — keeps the items source clean across re-runs.
Devil's Advocate
- Maybe
#if IOSwas deliberate to scope to iOS only. The PR explicitly claims Mac was tested, andItems2(no guard) does cover Mac — so the asymmetry between Items and Items2 is unintentional inconsistency, not deliberate scoping. - Maybe
UICollectionViewController.View.UserInteractionEnabled = falsewould already work. UIView'suserInteractionEnabled = falsedoes propagate to subviews — but the base mapper here doesn't actually set that on the outer wrapper in a way that survives Controller setup (the outer view's interaction was already left enabled by the controller infra). The bug existing in the wild confirms the base path doesn't work; setting it on the inner CV is the safest reliable fix. - Is the mapper shadowing actually a problem? Today, no. Tomorrow, when somebody adds a side effect to the base
MapIsEnabled, yes. WorthAppendToMappingor explicit base call.
Verdict: NEEDS_CHANGES
Confidence: high
Summary: The iOS fix shape is right and the gate confirms it works on iOS, but the legacy ItemsViewHandler change is silently MacCatalyst-blind due to #if IOS, while ItemsViewHandler2 covers Mac — that asymmetry plus the missing Android changes claimed in the PR description need to be resolved before merge. Switching the legacy registration to #if IOS || MACCATALYST (or moving it into ItemsViewHandler.iOS.cs) is a one-line fix that closes the platform gap and aligns the two handler variants.
### Root Cause When IsEnabled is set to false, CollectionView should not allow item selection. However, the IsEnabled state was not properly handled on both iOS and Android, allowing selection even when it was expected to be disabled. ### Description of Change * iOS: Disabled UserInteractionEnabled based on the IsEnabled state to prevent any user interaction. * Android: Disabled PlatformView and restricted touch interactions by overriding OnTouchEvent in MauiRecyclerView ### Issues Fixed <!-- Please make sure that there is a bug logged for the issue being fixed. The bug should describe the problem and how to reproduce it. --> Fixes #27770 **Tested the behavior in the following platforms.** - [x] Android - [x] Windows - [x] iOS - [x] Mac ### Output Screenshot | Before | After | |---------|--------| | **Android**<br> <video src="https://github.com/user-attachments/assets/e43ed3e8-5970-446a-a8bc-eb0de4e312d4" width="260" height="500"> |**Android**<br> <video src="https://github.com/user-attachments/assets/f8d1beef-445d-4c72-bc2c-2495f42ddfbd" width="260" height="500"> | | **iOS**<br> <video src="https://github.com/user-attachments/assets/659a7a38-72ae-4c93-8b3d-fed9137f0a50" width="300" height="600"> | **iOS**<br> <video src="https://github.com/user-attachments/assets/bb6fc013-bb5a-4d0e-85e7-2c0459aeea3b" width="300" height="600"> |
### Root Cause When IsEnabled is set to false, CollectionView should not allow item selection. However, the IsEnabled state was not properly handled on both iOS and Android, allowing selection even when it was expected to be disabled. ### Description of Change * iOS: Disabled UserInteractionEnabled based on the IsEnabled state to prevent any user interaction. * Android: Disabled PlatformView and restricted touch interactions by overriding OnTouchEvent in MauiRecyclerView ### Issues Fixed <!-- Please make sure that there is a bug logged for the issue being fixed. The bug should describe the problem and how to reproduce it. --> Fixes #27770 **Tested the behavior in the following platforms.** - [x] Android - [x] Windows - [x] iOS - [x] Mac ### Output Screenshot | Before | After | |---------|--------| | **Android**<br> <video src="https://github.com/user-attachments/assets/e43ed3e8-5970-446a-a8bc-eb0de4e312d4" width="260" height="500"> |**Android**<br> <video src="https://github.com/user-attachments/assets/f8d1beef-445d-4c72-bc2c-2495f42ddfbd" width="260" height="500"> | | **iOS**<br> <video src="https://github.com/user-attachments/assets/659a7a38-72ae-4c93-8b3d-fed9137f0a50" width="300" height="600"> | **iOS**<br> <video src="https://github.com/user-attachments/assets/bb6fc013-bb5a-4d0e-85e7-2c0459aeea3b" width="300" height="600"> |
### Root Cause When IsEnabled is set to false, CollectionView should not allow item selection. However, the IsEnabled state was not properly handled on both iOS and Android, allowing selection even when it was expected to be disabled. ### Description of Change * iOS: Disabled UserInteractionEnabled based on the IsEnabled state to prevent any user interaction. * Android: Disabled PlatformView and restricted touch interactions by overriding OnTouchEvent in MauiRecyclerView ### Issues Fixed <!-- Please make sure that there is a bug logged for the issue being fixed. The bug should describe the problem and how to reproduce it. --> Fixes #27770 **Tested the behavior in the following platforms.** - [x] Android - [x] Windows - [x] iOS - [x] Mac ### Output Screenshot | Before | After | |---------|--------| | **Android**<br> <video src="https://github.com/user-attachments/assets/e43ed3e8-5970-446a-a8bc-eb0de4e312d4" width="260" height="500"> |**Android**<br> <video src="https://github.com/user-attachments/assets/f8d1beef-445d-4c72-bc2c-2495f42ddfbd" width="260" height="500"> | | **iOS**<br> <video src="https://github.com/user-attachments/assets/659a7a38-72ae-4c93-8b3d-fed9137f0a50" width="300" height="600"> | **iOS**<br> <video src="https://github.com/user-attachments/assets/bb6fc013-bb5a-4d0e-85e7-2c0459aeea3b" width="300" height="600"> |
### Root Cause When IsEnabled is set to false, CollectionView should not allow item selection. However, the IsEnabled state was not properly handled on both iOS and Android, allowing selection even when it was expected to be disabled. ### Description of Change * iOS: Disabled UserInteractionEnabled based on the IsEnabled state to prevent any user interaction. * Android: Disabled PlatformView and restricted touch interactions by overriding OnTouchEvent in MauiRecyclerView ### Issues Fixed <!-- Please make sure that there is a bug logged for the issue being fixed. The bug should describe the problem and how to reproduce it. --> Fixes #27770 **Tested the behavior in the following platforms.** - [x] Android - [x] Windows - [x] iOS - [x] Mac ### Output Screenshot | Before | After | |---------|--------| | **Android**<br> <video src="https://github.com/user-attachments/assets/e43ed3e8-5970-446a-a8bc-eb0de4e312d4" width="260" height="500"> |**Android**<br> <video src="https://github.com/user-attachments/assets/f8d1beef-445d-4c72-bc2c-2495f42ddfbd" width="260" height="500"> | | **iOS**<br> <video src="https://github.com/user-attachments/assets/659a7a38-72ae-4c93-8b3d-fed9137f0a50" width="300" height="600"> | **iOS**<br> <video src="https://github.com/user-attachments/assets/bb6fc013-bb5a-4d0e-85e7-2c0459aeea3b" width="300" height="600"> |
### Root Cause When IsEnabled is set to false, CollectionView should not allow item selection. However, the IsEnabled state was not properly handled on both iOS and Android, allowing selection even when it was expected to be disabled. ### Description of Change * iOS: Disabled UserInteractionEnabled based on the IsEnabled state to prevent any user interaction. * Android: Disabled PlatformView and restricted touch interactions by overriding OnTouchEvent in MauiRecyclerView ### Issues Fixed <!-- Please make sure that there is a bug logged for the issue being fixed. The bug should describe the problem and how to reproduce it. --> Fixes #27770 **Tested the behavior in the following platforms.** - [x] Android - [x] Windows - [x] iOS - [x] Mac ### Output Screenshot | Before | After | |---------|--------| | **Android**<br> <video src="https://github.com/user-attachments/assets/e43ed3e8-5970-446a-a8bc-eb0de4e312d4" width="260" height="500"> |**Android**<br> <video src="https://github.com/user-attachments/assets/f8d1beef-445d-4c72-bc2c-2495f42ddfbd" width="260" height="500"> | | **iOS**<br> <video src="https://github.com/user-attachments/assets/659a7a38-72ae-4c93-8b3d-fed9137f0a50" width="300" height="600"> | **iOS**<br> <video src="https://github.com/user-attachments/assets/bb6fc013-bb5a-4d0e-85e7-2c0459aeea3b" width="300" height="600"> |
## What's Coming .NET MAUI inflight/candidate introduces significant improvements across all platforms with focus on quality, performance, and developer experience. This release includes 153 commits with various improvements, bug fixes, and enhancements. ## Activityindicator - [Android] Fix CollectionView ActivityIndicator not animating after header height change by @Vignesh-SF3580 in #35358 <details> <summary>🔧 Fixes</summary> - [CollectionView items fail to update ActivityIndicator state after header height change](#33780) </details> ## Animation - [Android] Fix Shadow property affecting transform matrix. by @Shalini-Ashokan in #32962 <details> <summary>🔧 Fixes</summary> - [[Android] Applying Shadow property affects the properties in Visual Transform Matrix](#32731) </details> ## API - Add delegate-based alert dialog extensibility convention (no public API changes) by @Redth in #35095 <details> <summary>🔧 Fixes</summary> - [Alert/Dialog system (`DisplayAlert`, `DisplayActionSheet`, `DisplayPromptAsync`) needs a public extensibility point](#34104) </details> ## Blazor - [Android] Fix for BlazorWebView predictive back callback blocks Android back-to-home animation by @BagavathiPerumal in #35538 <details> <summary>🔧 Fixes</summary> - [[Android] BlazorWebView predictive back callback blocks Android back-to-home animation](#35397) </details> - [Android] Fix BlazorWebView back callback can swallow the first Back press when its callback is stale-enabled by @devanathan-vaithiyanathan in #35611 <details> <summary>🔧 Fixes</summary> - [[inflight regression] Android BlazorWebView back callback can swallow the first Back press when its callback is stale-enabled](#35573) </details> ## Border - [Windows] Fixed the ContentView clip is not updated when wrapping inside the Border by @Ahamed-Ali in #30408 <details> <summary>🔧 Fixes</summary> - [[Windows] ContentView clip is not updated when wrapping inside the Border](#30404) </details> - Fix Border.StrokeDashArray leaks dashed Borders when using a shared Application resource by @devanathan-vaithiyanathan in #35544 <details> <summary>🔧 Fixes</summary> - [`Border.StrokeDashArray` leaks dashed Borders when using a shared Application resource](#35492) </details> - [Windows] Border: Add AutomationPeer support by @Vignesh-SF3580 in #35577 <details> <summary>🔧 Fixes</summary> - [Adding AutomationPeers to Windows Borders](#27627) </details> - [Windows] Fixed BoxView improper rendering inside Border by @Dhivya-SF4094 in #28465 <details> <summary>🔧 Fixes</summary> - [[Windows] Issues with BoxView Placement Inside Border](#19668) </details> ## Button - Prevent NullReferenceException in LayoutButton by @GamesAgeddon in #35284 <details> <summary>🔧 Fixes</summary> - [NullReferenceException on iOS in Button.LayoutButton from WrapperView.LayoutSubviews](#31048) </details> - Fix TextColor null reset to restore platform defaults on iOS and Android by @Shalini-Ashokan in #35563 <details> <summary>🔧 Fixes</summary> - [[Windows, Android, iOS & Mac]Button TextColor does not restore to platform default when reset to null after dynamic update](#35513) </details> ## CollectionView - Fix CollectionView grid spacing updates for first row and column by @KarthikRajaKalaimani in #34527 <details> <summary>🔧 Fixes</summary> - [[MAUI] I2_Vertical grid for horizontal Item Spacing and Vertical Item Spacing - horizontally updating the spacing only applies to the second column](#34257) </details> - [MacCatalyst] Fix CollectionView Header/Footer Not Expanding to Content Width by @KarthikRajaKalaimani in #35213 <details> <summary>🔧 Fixes</summary> - [[MacOS][CV2] I8_View header and footer_Horizontal_View - Footer on the right doesn't adapt when resizing the window](#35113) </details> - [iOS/MacCatalyst] Fix IndicatorView not updating when IndicatorSize is changed to default value by @Shalini-Ashokan in #35215 <details> <summary>🔧 Fixes</summary> - [[iOS/MacCatalyst] IndicatorView does not update when IndicatorSize is dynamically changed to the default value](#35214) </details> - CollectionView selecteditem background lost if collectionview (or parent) IsEnabled changed. by @KarthikRajaKalaimani in #31540 <details> <summary>🔧 Fixes</summary> - [CollectionView selecteditem background lost if collectionview (or parent) IsEnabled changed.](#20615) </details> - [iOS/macOS] CollectionView: Fix FlowDirection not working on EmptyView by @Dhivya-SF4094 in #32674 <details> <summary>🔧 Fixes</summary> - [[iOS, MacOS] FlowDirection not working on EmptyView in CollectionView](#32404) - [[iOS, Mac] CollectionView EmptyViewTemplate content text is mirrored when FlowDirection is RightToLeft](#34522) </details> - Fix iOS CollectionView stale layout invalidations by @filipnavara in #35245 <details> <summary>🔧 Fixes</summary> - [[iOS] CollectionView tries to invalidate cells with invalid indexes](#35244) </details> - Fix Android grouped CollectionView header/footer rebind leak by @AdamEssenmacher in #35368 <details> <summary>🔧 Fixes</summary> - [Memory leak when scrolling a CollectionView with IsGrouped=true](#17698) </details> - [Windows] Fix for Item should scrolled based on the GroupHeaderTemplate by @SuthiYuvaraj in #28074 <details> <summary>🔧 Fixes</summary> - [I9_Scroll by object for grouped data - The group name is always pined at the top after clicking 'Scroll to Proboscis Monkey' button](#27922) </details> - [Android] Fix ScrollTo regression when IsGrouped true on CollectionView by @SubhikshaSf4851 in #35356 <details> <summary>🔧 Fixes</summary> - [[10.0.60] ScrollTo(0) not working anymore on CollectionView when IsGrouped="True"](#35313) </details> - [Android] Fix CollectionView scrolling performance regression by @devanathan-vaithiyanathan in #35379 <details> <summary>🔧 Fixes</summary> - [[10.0.60] CollectionView scrolling performance regression](#35344) </details> - Optimize parent dynamic resource refresh by @AdamEssenmacher in #35408 <details> <summary>🔧 Fixes</summary> - [Memory usage increases when scrolling collectionview if resources count is more than 191](#22053) </details> - Fix CI failure for CollectionView Scrolling Feature Tests due to PR #35379 by @devanathan-vaithiyanathan in #35536 - [iOS & Mac] CarouselViewController2 leaks on iOS/MacCatalyst due to unremoved orientation notification observer by @SubhikshaSf4851 in #35532 <details> <summary>🔧 Fixes</summary> - [CarouselViewController2 leaks on iOS/MacCatalyst due to unremoved orientation notification observer](#35472) </details> - Fix CollectionView.SelectedItems leaks popped views when bound to a retained ObservableCollection by @HarishwaranVijayakumar in #35558 <details> <summary>🔧 Fixes</summary> - [`CollectionView.SelectedItems` leaks popped views when bound to a retained `ObservableCollection`](#35497) </details> - Fix for Android - Dynamic Updates to CollectionView Header/Footer and Templates Are Not Displayed by @SuthiYuvaraj in #28904 <details> <summary>🔧 Fixes</summary> - [Android - Dynamic Updates to CollectionView Header/Footer and Templates Are Not Displayed](#28676) </details> - [Windows] Fix CarouselView EmptyView display when filtering to zero items by @Shalini-Ashokan in #29247 <details> <summary>🔧 Fixes</summary> - [[Windows] [Scenario Day] EmptyView using Template displayed at the same time as the content](#7150) </details> - [Android/iOS] Fix IsEnabled=False on CollectionView not working by @devanathan-vaithiyanathan in #27749 <details> <summary>🔧 Fixes</summary> - [[iOS/Android] CollectionView IsEnabled Not Working](#27770) </details> - Fix CarouselView.Loop property does not update dynamically and fails to maintain the scroll position when the loop value is changed at runtime by @devanathan-vaithiyanathan in #29527 <details> <summary>🔧 Fixes</summary> - [[Android] CarouselView.Loop = false causes crash on Android when changed at runtime](#29411) - [Loop Binding in CarouselView Not Updating Dynamically at Runtime](#29449) </details> - [iOS / Mac] Fix CollectionView.ScrollTo(index) silently failing whenIsGrouped="True" by @Dhivya-SF4094 in #35609 <details> <summary>🔧 Fixes</summary> - [CollectionView.ScrollTo(index) doesn't work correctly when IsGrouped="True" on iOS, MacCatalyst, and Windows](#35326) </details> - Fix Android nested carousel scrolling by @AdamEssenmacher in #35656 <details> <summary>🔧 Fixes</summary> - [Vertical scrolling not working for CarouselView and CustomLayouts](#7814) </details> - [Inflight regression] Fixed Test failures ModalTabbedPagePushAsyncShouldOverlayBottomNavigationView and GroupedCollectionViewScrollToIndexScrollsToCorrectItem by @Dhivya-SF4094 in #35823 - Fix CarouselView tests fail in June 8 Candidate by @devanathan-vaithiyanathan in #35825 ## Core - Reduce allocations on AnimationManager by @pictos in #35612 <details> <summary>🔧 Fixes</summary> - [AnimationManager is allocating a lot](#35654) </details> ## Core Lifecycle - Fix device test memory by @pictos in #35487 <details> <summary>🔧 Fixes</summary> - [Memory leak Device.Test pass with false positive](#35485) </details> ## Datepicker - Fix MacCatalyst DatePicker focus handling by @AdamEssenmacher in #35553 <details> <summary>🔧 Fixes</summary> - [[mauipalooza] DatePicker focus only works first time](#5947) </details> ## DateTimePicker - [Android] Fix DatePicker dialog dismisses after the device is rotated by @HarishwaranVijayakumar in #34980 <details> <summary>🔧 Fixes</summary> - [[Android] [Regression] DatePicker dialog dismisses after the device is rotated](#34973) </details> ## Docs - doc: Add paragraph to README.md explaining how to fetch the `maui` project templates by @durandt in #34561 ## Drawing - [Android] Fix LinearGradientBrush rendering as opaque black box by @SubhikshaSf4851 in #35299 <details> <summary>🔧 Fixes</summary> - [[Regression] LinearGradientBrush broken on Android in 10.0.60](#35280) - [10.0.60 breaks transparency on Brushes (on Android?)](#35354) </details> - Fix polygon points collection handler leak by @AdamEssenmacher in #35526 <details> <summary>🔧 Fixes</summary> - [PolygonHandler and PolylineHandler leak when Points is replaced before disconnect](#35387) </details> ## Editor - [iOS] Fix Editor losing scrollability after rotation when CharacterSpacing is applied by @Vignesh-SF3580 in #35309 <details> <summary>🔧 Fixes</summary> - [[.NET 10][iOS] D2 - Editor can't be scrolled after rotating simulator.](#35114) </details> - [Inflight/Candidate][iOS & Mac] Fix for Editor height inconsistency when VerticalTextAlignment is Center or End on iOS and MacCatalyst by @BagavathiPerumal in #35662 <details> <summary>🔧 Fixes</summary> - [[MAUI] D13_Customize_Text_Alignment - Text Editor Height is not consistent](#35615) </details> ## Entry - [iOS/Mac] Fix Entry clear button retaining tint color after TextColor is reset to null by @SyedAbdulAzeemSF4852 in #35177 <details> <summary>🔧 Fixes</summary> - [[iOS/Mac]Entry ClearButtonVisibility color does not reset when TextColor is set to null](#35076) </details> - [iOS/MacCatalyst] Fix Entry clear button appearing dimmed compared to TextColor by @SyedAbdulAzeemSF4852 in #35541 <details> <summary>🔧 Fixes</summary> - [[MacCatalyst] [Entry] ClearButtonVisibility color appears dimmed compared to TextColor](#35517) </details> - Fix pill-shaped focus ring on macOS 26 by @Dhivya-SF4094 in #35393 <details> <summary>🔧 Fixes</summary> - [.Net 10 Picker item not centered and wrong focus outline of Entry on Mac](#34899) </details> - Fix Entry select all text on refocus not working on WinUI by @kubaflo in #35383 ## Essentials - [Android] Fix Capture video crashes after stopping recording on Android 12 by @HarishwaranVijayakumar in #35638 <details> <summary>🔧 Fixes</summary> - [Capture video crashes after stopping recording on Android 12](#28891) </details> - [Essentials] Browser.OpenAsync(External): drop visibility-filtered ResolveActivity pre-check by @Kebechet in #35652 <details> <summary>🔧 Fixes</summary> - [Browser.OpenAsync(External) on Android throws FeatureNotSupportedException for verified App Link owner URLs even with documented <queries> fix applied](#35651) </details> ## Essentials Texttospeech - [Mac, iOS, Windows] Fix for inconsistent Text-to-Speech rate behavior by @HarishwaranVijayakumar in #32850 <details> <summary>🔧 Fixes</summary> - [[Essentials] TTS rate](#32492) </details> ## Flyoutpage - [iOS/Mac] Fix FlyoutPage RTL FlowDirection is not working by @devanathan-vaithiyanathan in #34831 <details> <summary>🔧 Fixes</summary> - [[iOS/Mac] FlyoutPage RTL FlowDirection is not working properly](#34830) </details> - [Android] Fix for Android 16 Back button is not working after command from FlyoutPage by @BagavathiPerumal in #35196 <details> <summary>🔧 Fixes</summary> - [Android: BackButton on Android 16 not working after command from FlyOutPage](#33508) </details> ## Gestures - Fix DragGestureRecognizer.DropCompleted event not firing in Android platform by @KarthikRajaKalaimani in #35179 <details> <summary>🔧 Fixes</summary> - [[Android] DragGestureRecognizer.DropCompleted event not firing](#17554) </details> - Windows: Ensure layouts without background participate in hit testing by @jpd21122012 in #34364 <details> <summary>🔧 Fixes</summary> - [[Windows] TapGestureRecognizer does NOT work on a ContentView without Background](#32279) </details> - [iOS] Fix VoiceOver dropping child labels on layouts with SemanticProperties.Hint or TapGestureRecognizer by @Vignesh-SF3580 in #35590 <details> <summary>🔧 Fixes</summary> - [[iOS] VoiceOver does not correctly describe View with GestureRecognizers](#34380) </details> ## Hybridwebview - Fix RemovePossibleQueryString to also strip URL fragments by @kubaflo in #35551 <details> <summary>🔧 Fixes</summary> - [HybridWebViewQueryStringHelper.RemovePossibleQueryString removes '?' but not other special characters e.g. '#'](#31472) </details> - [Revert] - [Windows] Fix WebView blank rendering when used with HybridWebView by @SubhikshaSf4851 in #35814 ## Image - Avoid image source layout invalidation for fixed-size views by @AdamEssenmacher in #35369 <details> <summary>🔧 Fixes</summary> - [Image source swaps thrash layout under fixed constraints, tanking frame rate when scrolling virtualized collections](#32457) </details> - [Windows] Fix Image layout inconsistency caused by async decode race in GetDesiredSize by @praveenkumarkarunanithi in #34699 <details> <summary>🔧 Fixes</summary> - [[Windows] Image cropping produces inconsistent results when window is minimized or resized](#32393) </details> - [Testing] Include more testing around Windows Image Aspect recent fixes by @kubaflo in #35620 <details> <summary>🔧 Fixes</summary> - [[Testing] Include more testing around Windows Image Aspect recent fixes](#31686) </details> - Revert PR #30068 — Fix FontImageSource centering regression on Windows by @Shalini-Ashokan in #35642 <details> <summary>🔧 Fixes</summary> - [[Windows] Image with FontImageSource is not centered and gets clipped when WidthRequest/HeightRequest equals FontImageSource Size](#35618) </details> - [Android] Fix screenshot from WebView content not working by @kubaflo in #35384 <details> <summary>🔧 Fixes</summary> - [[Android] Loading the captured screenshot from webview content to Image control does not visible](#30010) </details> ## Label - Improve label mapping performance and ensure complete coverage including ToPlatform and subsequent property changes by @Tamilarasan-Paranthaman in #31159 - Fix for Label.FormattedText leaks labels when shared FormattedString is stored in Application.Resources by @BagavathiPerumal in #35582 <details> <summary>🔧 Fixes</summary> - [`Label.FormattedText` leaks labels when shared `FormattedString` is stored in `Application.Resources`](#35495) </details> - [iOS] Fix Label Span formatting test failures on candidate branch by @Vignesh-SF3580 in #35815 ## Layout - [iOS, Mac] Fix Item spacing not properly applied between items in Horizontal LinearItemsLayout by @Dhivya-SF4094 in #35445 <details> <summary>🔧 Fixes</summary> - [[CollectionView2] Item spacing not properly applied between items in Horizontal LinearItemsLayout](#35429) </details> - [Windows] Add Automation Id support for Layouts. by @SubhikshaSf4851 in #35562 <details> <summary>🔧 Fixes</summary> - [[Windows] AutomationId does not work for ContentView, Layouts and controls that inherit them](#4715) </details> - Avoid layout diagnostics allocations without listeners by @AdamEssenmacher in #35475 <details> <summary>🔧 Fixes</summary> - [MAUI 10 layout diagnostics no-consumer path is not zero-allocation](#35473) </details> - [Windows/Android] FlexLayout: Fix wrap misalignment due to floating-point precision by @SuthiYuvaraj in #31341 <details> <summary>🔧 Fixes</summary> - [FlexLayout Wrap Misalignment with Dynamically-Sized Buttons in .NET MAUI](#30957) </details> ## Listview - Fix Binding for ListView.IsRefreshing by @bill2004158 in #28516 <details> <summary>🔧 Fixes</summary> - [Bind ListView.IsRefreshing is not work.](#28514) </details> ## Map - Fix iOS/Catalyst MapPool retention with MapElements by @AdamEssenmacher in #35480 <details> <summary>🔧 Fixes</summary> - [iOS/Mac Catalyst MapHandler leaks MAUI Map views and MapElements through MapPool](#35479) </details> - Fix Android map view lifecycle cleanup by @AdamEssenmacher in #35476 <details> <summary>🔧 Fixes</summary> - [Navigating to a page with Maps multiple times Increase RAM Usage but doesn't reduce it back after navigating back](#15257) </details> - Fix Android map element options retention by @AdamEssenmacher in #35634 <details> <summary>🔧 Fixes</summary> - [[Regression] [Android] [Maps] Map locks up after rendering 50 Polylines](#20502) </details> ## Menubar - [MacCatalyst] Fix KeyboardAccelerator with Cmd+Shift modifiers breaks entire MenuBarItem on Mac Catalyst by @KarthikRajaKalaimani in #35318 <details> <summary>🔧 Fixes</summary> - [[Bug] KeyboardAccelerator with Cmd+Shift modifiers breaks entire MenuBarItem on Mac Catalyst](#35279) </details> ## Navigation - [iOS, Mac] Fix OnBackButtonPressed not invoked for NavigationPage and Shell by @Dhivya-SF4094 in #35072 <details> <summary>🔧 Fixes</summary> - [On Screen Back Button Does Not Fire OnBackButtonPressed in Android](#9095) - [ContentPage's OnBackButtonPressed not invoked on iOS and MacCatalyst](#8296) </details> - Fix Android stale ContainerView root leak by @AdamEssenmacher in #35372 <details> <summary>🔧 Fixes</summary> - [Android: Stale ContainerView retains replaced FlyoutPage graph](#35371) </details> - [Android] Fix for predictive back-to-home animation blocked by unconditional back callback registration by @BagavathiPerumal in #35223 <details> <summary>🔧 Fixes</summary> - [OnBackInvokedCallbacks block back-to-home animation](#34594) - [Migrate to OnBackPressedCallback](#24752) </details> - Revert [Android, iOS] - Flyout icon should remain visible when a page is pushed onto a NavigationPage or Shell page with the back button disabled. by @praveenkumarkarunanithi in #35604 ## Picker - [iOS] Fix Picker CharacterSpacing lost after item selection when Title is set by @SyedAbdulAzeemSF4852 in #34974 <details> <summary>🔧 Fixes</summary> - [[iOS] Picker loses CharacterSpacing after item selection when Title is set](#34971) </details> - [iOS] Fix Picker CharacterSpacing ignored on initial load by @SyedAbdulAzeemSF4852 in #34957 <details> <summary>🔧 Fixes</summary> - [[iOS] Picker ignores CharacterSpacing on initial load](#34955) </details> - [Windows] Fix for Picker CharacterSpacing Not Being Applied to Title and Dropdown Items by @SyedAbdulAzeemSF4852 in #30612 <details> <summary>🔧 Fixes</summary> - [[Windows] Picker CharacterSpacing property not applied to Title and PickerItems text](#30464) </details> - Fix Picker SelectedIndex deferred initialization by @AdamEssenmacher in #35629 <details> <summary>🔧 Fixes</summary> - [Picker Attribute "SelectedIndex" Not being respected on page load on Android?](#9150) </details> ## Progressbar - Fix iOS ProgressBar bounding box by @AdamEssenmacher in #35507 <details> <summary>🔧 Fixes</summary> - [[iOS] ProgressBar and Label don't correctly obey height and width at the core level](#7935) </details> ## RadioButton - [Windows, Android] Fix Border Color and Border Width Not applying for Radio Button by @HarishwaranVijayakumar in #35616 <details> <summary>🔧 Fixes</summary> - [[Windows, Android] Border Color and Border Width Not applying for Radio Button.](#35587) </details> - [inflight/current] Fixes a CS0111 build failure in RadioButton.cs caused by a duplicate OnPropertyChanged override by @HarishwaranVijayakumar in #35631 - Revert - Fix TalkBack not correctly narrating RadioButtons with Content by @devanathan-vaithiyanathan in #35625 <details> <summary>🔧 Fixes</summary> - [[Android] MissingMethodException AccessibilityNodeInfoCompat.set_Checked(bool) on 10.0.70 due to AndroidX.Core 1.17 breaking change](#35584) </details> ## Refreshview - [Windows] Fix RefreshView IsRefreshing property not working while binding by @devanathan-vaithiyanathan in #34845 <details> <summary>🔧 Fixes</summary> - [[Windows] RefreshView IsRefreshing property not working while binding](#30535) </details> - [Android] Fix for RefreshView triggering pull-to-refresh when scrolling inside a WebView with internal scrollable content by @BagavathiPerumal in #34614 <details> <summary>🔧 Fixes</summary> - [[Android] RefreshView triggers pull-to-refresh immediately when scrolling up inside a WebView](#33510) </details> ## SafeArea - [Android] Fix bottom safe area padding dropping to zero when keyboard is shown by @praveenkumarkarunanithi in #35084 <details> <summary>🔧 Fixes</summary> - [[Android] Bottom insets issues when keyboard is shown.](#32871) </details> - Gate SafeArea inset listeners in recycler items by @AdamEssenmacher in #35664 <details> <summary>🔧 Fixes</summary> - [[10.0.60] CollectionView scrolling performance regression](#35344) </details> ## ScrollView - [Windows] Fix COMException when restoring a ScrollView as ContentPage.Content after swapping it out by @Vignesh-SF3580 in #35360 <details> <summary>🔧 Fixes</summary> - [COMException when clone a page's content to a object and set it back later in mainthread on Windows](#35277) </details> - Fix - ScrollView.ScrollToAsync(x, y, animated) doesn't work when called from Page.OnAppearing by @Shalini-Ashokan in #35395 <details> <summary>🔧 Fixes</summary> - [[iOS] ScrollView.ScrollToAsync(x, y, animated) doesn't work when called from Page.OnAppearing](#31177) </details> ## Searchbar - [Android] Fix SearchBar IME full-screen extract mode in landscape orientation by @SubhikshaSf4851 in #35197 <details> <summary>🔧 Fixes</summary> - [[Android] Investigate SearchBar presentation in horizontal screen orientation ](#14708) </details> - [iOS 26] Fix SearchBar layout spacing issues for small HeightRequest values by @devanathan-vaithiyanathan in #35347 <details> <summary>🔧 Fixes</summary> - [Spacing problem with maui 10.0.60 iOS](#35286) </details> ## SearchBar - [Windows] Fix SearchHandler does not focus when ShowSoftInputAsync is called by @praveenkumarkarunanithi in #35079 <details> <summary>🔧 Fixes</summary> - [[Windows] SearchHandler.ShowSoftInputAsync() does not focus the SearchHandler](#34930) </details> ## Shell - Fix Android layout jump when navigating with IME open and NavBarIsVisible=false by @jpd21122012 in #34621 <details> <summary>🔧 Fixes</summary> - [Shell page without NavBar jumping when navigating with keyboard open](#34584) </details> - [Android] Add defensive not null check to SearchHandlerAppearanceTracker.FocusChange by @Transis-Felipe in #29939 - [Android] Fix for Shell colors change before navigation completes on Android in .NET 10 by @BagavathiPerumal in #35295 <details> <summary>🔧 Fixes</summary> - [Shell colors change before navigation completes on Android in .NET 10](#35060) </details> - [Windows] Fix Shell FlyoutItem not taking full width by @SubhikshaSf4851 in #35131 <details> <summary>🔧 Fixes</summary> - [MAUI WinUI Grids don't render properly in flyout menu](#19542) - [[Windows] [.NET 8 RC2] FlyoutItem Backgroundcolor Is not fully displaying](#18238) </details> - [Android, iOS, Catalyst] Fix SearchHandler.BackgroundColor cannot be reset to null by @HarishwaranVijayakumar in #35224 <details> <summary>🔧 Fixes</summary> - [[Android, iOS, Catalyst] SearchHandler.BackgroundColor cannot be reset to null](#35088) </details> - Fix for ApplyQueryAttributes being called on non-destination pages during back navigation by @BagavathiPerumal in #35392 <details> <summary>🔧 Fixes</summary> - [ApplyQueryAttributes gets called for not activated (navigated to) page on back](#35183) </details> - [Android] Fix Shell flyout background to follow Material 3 theme colors by @SyedAbdulAzeemSF4852 in #35148 <details> <summary>🔧 Fixes</summary> - [[Android] Shell Flyout ignores Material 3 surface color when UseMaterial3 is enabled](#35147) </details> - [Android] Fix Shell.FlyoutHeader background incorrect by @SyedAbdulAzeemSF4852 in #35489 <details> <summary>🔧 Fixes</summary> - [[Android] Shell.FlyoutHeader background is incorrect](#35416) </details> - [iOS/MacCatalyst] Fix Shell.BackgroundColor not applied to bottom TabBar by @Shalini-Ashokan in #35545 <details> <summary>🔧 Fixes</summary> - [[MacCatalyst] Shell.BackgroundColor not applied to bottom TabBar](#35380) - [[Catalyst] Shell.TabBarBackgroundColor is not applied](#35381) </details> - [Android] Fix Shell FlyoutIcon tint loss after navigation by @SyedAbdulAzeemSF4852 in #35561 <details> <summary>🔧 Fixes</summary> - [[Android] The flyout icon loses colours](#35390) </details> - [iOS] Fix Shell - opened keyboard on modal page shifts parent page/frame behind modal after update to 10.0.60 by @KarthikRajaKalaimani in #35559 <details> <summary>🔧 Fixes</summary> - [[iOS] Shell - opened keyboard on modal page shifts parent page/frame behind modal after update to 10.0.60](#35401) </details> - Fix intermediate pages not receiving query parameters in multi-page Shell navigation by @mattleibow in #35432 <details> <summary>🔧 Fixes</summary> - [Shell GoToAsync: no way to pass query parameters to intermediate pages in multi-segment navigation](#35107) </details> - [Windows] Fix Shell title bar overlap with window controls in RTL mode by @Shalini-Ashokan in #33109 <details> <summary>🔧 Fixes</summary> - [[Windows] Binding RTL FlowDirection in Shell causes Flyout MenuIcon and native window controls to overlap](#32476) </details> - [macOS] Fix IsEnabled property false not working on MenuBarItem by @devanathan-vaithiyanathan in #35546 <details> <summary>🔧 Fixes</summary> - [[macOS] IsEnabled property false not working on MenuBarItem](#34038) </details> - Fix Android Shell top inset when nav bar is hidden by @ne0rrmatrix in #35555 <details> <summary>🔧 Fixes</summary> - [wrong statusbar height when Android device has a notch](#35103) </details> - Fix Changing Content property of ShellContent doesn't change visual content by @devanathan-vaithiyanathan in #34630 <details> <summary>🔧 Fixes</summary> - [Changing Content property of ShellContent doesn't change visual content. ](#12669) </details> - Fixed a NullReferenceException when starting application with empty shell on Windows by @Shalini-Ashokan in #28879 <details> <summary>🔧 Fixes</summary> - [NullReferenceException when starting application with empty shell on Windows](#21562) - [Using SelectionChangedCommand with CollectionView in Shell.FlyoutContent results in Win32 Unhandled Exception](#10041) </details> ## Slider - [iOS] Slider: Scale ThumbImageSource to match default thumb size by @NirmalKumarYuvaraj in #34184 <details> <summary>🔧 Fixes</summary> - [[Slider] MAUI Slider thumb image is big on android](#13258) </details> ## Stepper - Fix iOS 26 Stepper overlap in landscape by @AdamEssenmacher in #35374 <details> <summary>🔧 Fixes</summary> - [[.NET10] D10-The number and buttons overlap after rotating the simulator.](#35211) </details> ## SwipeView - Fix SwipeViews with invoked properties crash the app in Release mode by @BagavathiPerumal in #35208 <details> <summary>🔧 Fixes</summary> - [[iOS/Catalyst] Swipeviews with invoked properties crash the app in Release](#18055) </details> - Fix SwipeItemView command leak by @AdamEssenmacher in #35510 <details> <summary>🔧 Fixes</summary> - [`SwipeItemView.Command` leaks row views and command parameters through `CanExecuteChanged`](#35498) </details> - [iOS/Android] Fix SwipeItem.IsVisible not refreshing native swipe items when binding changes by @SyedAbdulAzeemSF4852 in #35217 <details> <summary>🔧 Fixes</summary> - [SwipeItem.IsVisible doesn't properly refresh the native swipe items when the binding value changes dynamically](#34832) </details> - Fix SwipeView memory leak when SwipeItems are reused or replaced by @Vignesh-SF3580 in #35539 <details> <summary>🔧 Fixes</summary> - [SwipeView leaks when SwipeItems are reused or replaced](#35481) </details> - Fix SwipeItem IconImageSource color handling and rendering across platforms by @Shalini-Ashokan in #35632 <details> <summary>🔧 Fixes</summary> - [[Android] SwipeItem IconImageSource should allow more configuration](#23074) </details> ## Switch - [Android/Windows] Fix RadioButton gradient not clearing when switching background by @Shalini-Ashokan in #34997 <details> <summary>🔧 Fixes</summary> - [RadioButton Background does not reset when set to null at runtime](#34993) </details> - [Windows] Fix "PlatformView cannot be null here" exception during handler disconnect by @kubaflo in #35314 <details> <summary>🔧 Fixes</summary> - ["PlatformView cannot be null here" Exception in Switch control [Windows]](#27101) </details> - [iOS 26] Fix Switch ThumbColor and OffColor not applied on initial load by @SyedAbdulAzeemSF4852 in #35400 <details> <summary>🔧 Fixes</summary> - [iOS 26 Switch default color for Off and On is incorrect + Off Color is not applied at start + Thumb Colors is not applied](#35257) </details> - [Android] Fix AppBar flicker on CheckBox/Switch toggle with Material 3 by @Dhivya-SF4094 in #35181 <details> <summary>🔧 Fixes</summary> - [[Android] AppBar flicker while changing the CheckBox or Switch state after scrolling in Material 3](#35180) </details> - [Android] Fix Switch Shadow Does Not Follow Thumb when Toggle On or Off by @Dhivya-SF4094 in #35623 <details> <summary>🔧 Fixes</summary> - [[Android] Switch Shadow Does Not Follow Thumb when Toggle On or Off](#30046) </details> ## TabbedPage - [Android] Fix TabbedPage truncating tab titles instead of scrolling by @Shalini-Ashokan in #35086 <details> <summary>🔧 Fixes</summary> - [Maui migrating Xamarin to Maui - Tabbed Page Scroll Issue - Tabs are not scrolling](#16470) </details> - [Android] Fix BottomNavigationView remaining visible for TabbedPage inside modal NavigationPage after PushAsync by @Dhivya-SF4094 in #35359 <details> <summary>🔧 Fixes</summary> - [Android TabbedPage inside Modal Navigation does not overlay BottomNavigationView after PushAsync in .NET MAUI 10.0.60](#35331) </details> - [Android & iOS] TabbedPage leaks with shared GradientBrush. by @SubhikshaSf4851 in #35543 <details> <summary>🔧 Fixes</summary> - [TabbedPage leaks renderer/manager when BarBackground uses shared GradientBrush resource](#35469) </details> ## Templates - Bumps Syncfusion.Maui.Toolkit dependency to version 1.0.10 by @PaulAndersonS in #35608 ## Toolbar - Fix Android app bar inset background coloring by @ne0rrmatrix in #35601 <details> <summary>🔧 Fixes</summary> - [Android Edge-to-Edge: Shell and NavigationPage Top Bar colour is not used for status bar.](#35568) </details> ## Tooling - Add default .gitignore to MAUI project templates by @davidortinau in #34862 <details> <summary>🔧 Fixes</summary> - [Add a gitignore file to the Maui template in VS 2022](#4131) </details> - Fix: Propagate AdditionalProperties from ProjectReference in ResizetizeCollectItems by @mattleibow in #35575 <details> <summary>🔧 Fixes</summary> - [Resizetizer GetMauiItems does not propagate ProjectReference AdditionalProperties](#35574) </details> ## WebView - [Windows] Fix WebView blank rendering when used with HybridWebView by @SubhikshaSf4851 in #35092 <details> <summary>🔧 Fixes</summary> - [[Windows] WebView Regression from NET9 to NET10](#34558) </details> - Fix AOT integration test failures: suppress IL3050/IL2026 for HybridWebViewHandler in AddControlsHandlers by @mattleibow via @Copilot in #34868 - Fix Android activity result callback leak by @AdamEssenmacher in #35436 <details> <summary>🔧 Fixes</summary> - [Android WebView file chooser callbacks leak via ActivityResultCallbackRegistry](#35405) </details> - [Windows] Fix WebView Does Not Inherit App Theme by @devanathan-vaithiyanathan in #35037 <details> <summary>🔧 Fixes</summary> - [WebView on Windows Does Not Inherit App Theme](#34823) </details> - Fix for WebView leaks when reusing a shared WebViewSource by @BagavathiPerumal in #35524 <details> <summary>🔧 Fixes</summary> - [WebView leaks when reusing a shared WebViewSource](#35483) </details> - Destroy Android WebView on handler disconnect by @AdamEssenmacher in #35552 <details> <summary>🔧 Fixes</summary> - [Right way to dispose page with WebView](#18021) </details> ## Xaml - Fix: Enable VisualStateManager to set Style property dynamically by @Shalini-Ashokan in #33389 <details> <summary>🔧 Fixes</summary> - [Setting the `Style` property using the `VisualStateManager` within a Style resource does not work](#17175) </details> - Fix Implicit parameter conversion from integer to byte fails with source generated XAML by @KarthikRajaKalaimani in #35444 <details> <summary>🔧 Fixes</summary> - [Implicit parameter conversion from integer to byte fails with source generated XAML](#35396) </details> <details> <summary>🔧 Infrastructure (3)</summary> - Fix: Build fails when appicon is an empty (but valid) SVG by @Shalini-Ashokan in #35305 <details> <summary>🔧 Fixes</summary> - [Build fails when appicon is an empty (but valid) svg after upgrade to 10.0.60](#35293) </details> - [inflight/current] Fix CS0111 duplicate GetNativeCharacterSpacing in PickerHandlerTests.iOS by @SyedAbdulAzeemSF4852 in #35419 - Update WinAppSDK to 1.8.260508005 by @kubaflo in #35678 </details> <details> <summary>🧪 Testing (3)</summary> - Backport Test Fixes and Snapshots from SR to Inflight Branch by @Tamilarasan-Paranthaman in #35499 - Fix hardcoded version of Microsoft.DotNet.XHarness.TestRunners.Xunit in test projects by @akoeplinger in #29905 - [Testing] Fixed Build error on inflight/ candidate PR 35716 by @HarishKumarSF4517 in #35730 </details> <details> <summary>🏠 Housekeeping (1)</summary> - [HouseKeeping] Fix inconsistant namespace in HostApp by @NirmalKumarYuvaraj in #35210 </details> <details> <summary>📦 Other (8)</summary> - Add .cab and ReconnectModal.razor.js to signing config by @jesuszarate in #35026 - Fix typo in Clipboard.shared.cs by @Deadpikle in #35316 - Fix single modifier for NSMenuItem accelerators by @jeremy-visionaid in #35351 - Avoid unnecessary LINQ enumerations by @jeremy-visionaid in #35272 - [Testing] Replace retryDelay with retryTimeout in UI tests by @kubaflo in #35367 - Replace JavaFinalize() with Dispose(bool) in GenericAnimatorListener by @jonathanpeppers in #35548 - Fix incorrect SDK provisioning commands in integration-tests instructions by @davidnguyen-tech in #34992 - Fix VisualElement.ChangeVisualState() gets stuck in Selected state by @Dhivya-SF4094 in #35421 <details> <summary>🔧 Fixes</summary> - [VisualElement's ChangeVisualState gets stuck in Selected state](#35399) </details> </details> <details> <summary>📝 Issue References</summary> Fixes #4131, Fixes #4715, Fixes #5947, Fixes #7150, Fixes #7814, Fixes #7935, Fixes #8296, Fixes #9095, Fixes #9150, Fixes #10041, Fixes #12669, Fixes #13258, Fixes #14708, Fixes #15257, Fixes #16470, Fixes #17175, Fixes #17554, Fixes #17698, Fixes #18021, Fixes #18055, Fixes #18238, Fixes #19542, Fixes #19668, Fixes #20502, Fixes #20615, Fixes #21562, Fixes #22053, Fixes #23074, Fixes #24752, Fixes #27101, Fixes #27627, Fixes #27770, Fixes #27922, Fixes #28514, Fixes #28676, Fixes #28891, Fixes #29411, Fixes #29449, Fixes #30010, Fixes #30046, Fixes #30404, Fixes #30464, Fixes #30535, Fixes #30957, Fixes #31048, Fixes #31177, Fixes #31472, Fixes #31686, Fixes #32279, Fixes #32393, Fixes #32404, Fixes #32457, Fixes #32476, Fixes #32492, Fixes #32731, Fixes #32871, Fixes #33508, Fixes #33510, Fixes #33780, Fixes #34038, Fixes #34104, Fixes #34257, Fixes #34380, Fixes #34522, Fixes #34558, Fixes #34584, Fixes #34594, Fixes #34823, Fixes #34830, Fixes #34832, Fixes #34899, Fixes #34930, Fixes #34955, Fixes #34971, Fixes #34973, Fixes #34993, Fixes #35060, Fixes #35076, Fixes #35088, Fixes #35103, Fixes #35107, Fixes #35113, Fixes #35114, Fixes #35147, Fixes #35180, Fixes #35183, Fixes #35211, Fixes #35214, Fixes #35244, Fixes #35257, Fixes #35277, Fixes #35279, Fixes #35280, Fixes #35286, Fixes #35293, Fixes #35313, Fixes #35326, Fixes #35331, Fixes #35344, Fixes #35354, Fixes #35371, Fixes #35380, Fixes #35381, Fixes #35387, Fixes #35390, Fixes #35396, Fixes #35397, Fixes #35399, Fixes #35401, Fixes #35405, Fixes #35416, Fixes #35429, Fixes #35469, Fixes #35472, Fixes #35473, Fixes #35479, Fixes #35481, Fixes #35483, Fixes #35485, Fixes #35492, Fixes #35495, Fixes #35497, Fixes #35498, Fixes #35513, Fixes #35517, Fixes #35568, Fixes #35573, Fixes #35574, Fixes #35584, Fixes #35587, Fixes #35615, Fixes #35618, Fixes #35651, Fixes #35654 </details> **Full Changelog**: main...inflight/candidate
Root Cause
When IsEnabled is set to false, CollectionView should not allow item selection. However, the IsEnabled state was not properly handled on both iOS and Android, allowing selection even when it was expected to be disabled.
Description of Change
Issues Fixed
Fixes #27770
Tested the behavior in the following platforms.
Output Screenshot
Android-Before.mov
Android-After.mov
iOS-Before.mov
iOS-After.mov