[iOS][CV2] Fix page can be dragged down, and it would cause an extra space between Header and EmptyView text by devanathan-vaithiyanathan · Pull Request #31840 · dotnet/maui

devanathan-vaithiyanathan · 2025-10-01T13:49:11Z

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Issue Details

On iOS, a CollectionView with Header, EmptyView shows incorrect spacing when dragging down. The EmptyView shifts downward by double its intended Y offset leaving extra space between the header and empty view.

Root Cause:

The extra spacing happens because the header offset is applied twice. On iOS, the native container (_emptyUIView) is shifted by the ScrollView during header space calculations, and at the same time the MAUI element inside that container is also arranged with the same offset. As a result, the element ends up positioned once by its container and again by MAUI, which doubles the intended Y offset.

Description of changes:

Previously, the MAUI element was being arranged using absolute coordinates of the frame via _emptyViewFormsElement.Arrange(frame.ToRectangle()), which caused the header offset to be applied twice — once by the native container and once by the MAUI element.

The fix changes the arrange call to _emptyViewFormsElement.Arrange(new Rect(0, 0, frame.Width, frame.Height)). This positions the MAUI element at the origin (0,0) within the native container while retaining its width and height.

Issues Fixed

Fixes #31465

Tested the behavior in the following platforms.

Android
Windows
iOS
Mac

Before	After
iOS Before.mov	iOS After.mov

jsuarezruiz · 2025-10-02T12:36:07Z

/azp run

azure-pipelines · 2025-10-02T12:36:32Z

Azure Pipelines successfully started running 3 pipeline(s).

jsuarezruiz · 2025-10-02T12:38:59Z

src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue31465.cs

+	{
+		App.WaitForElement("HeaderLabel");
+		App.Click("CollectionViewHeader");
+		VerifyScreenshot();


Pending snapshots, running a build.

@jsuarezruiz , I have added the snapshots for all platforms

Copilot

Pull Request Overview

This PR fixes an iOS-specific issue where CollectionView's EmptyView appears with doubled Y offset when the page is dragged down, creating unwanted extra space between the Header and EmptyView.

Changes the EmptyView positioning logic to use relative coordinates instead of absolute coordinates
Adds comprehensive UI tests to verify the fix across platforms
Ensures EmptyView is positioned correctly within its native container without double-offsetting

Reviewed Changes

Copilot reviewed 3 out of 7 changed files in this pull request and generated 1 comment.

File	Description
src/Controls/src/Core/Handlers/Items2/iOS/ItemsViewController2.cs	Fixed EmptyView positioning to use relative coordinates (0,0) instead of absolute frame coordinates
src/Controls/tests/TestCases.HostApp/Issues/Issue31465.cs	Added test page with CollectionView, Header, and EmptyView to reproduce the positioning issue
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue31465.cs	Added automated UI test to verify EmptyView positioning behavior

Copilot · 2025-10-03T11:19:09Z

src/Controls/tests/TestCases.HostApp/Issues/Issue31465.cs

+                    TextColor = Colors.Black,
+                    FontSize = 16,
+                    // This binding works with string items
+                    BindingContext = "{Binding .}" 


The BindingContext property should not be assigned a string value. It should be assigned the actual binding expression. Use this.SetBinding(BindingContextProperty, \".\") instead, or remove this line entirely since the binding context will be set automatically for DataTemplate items.

Suggested change

BindingContext = "{Binding .}"

jsuarezruiz · 2025-10-06T05:49:11Z

/azp run MAUI-UITests-public

azure-pipelines · 2025-10-06T05:49:22Z

Azure Pipelines successfully started running 1 pipeline(s).

jsuarezruiz · 2025-10-06T07:25:23Z

src/Controls/src/Core/Handlers/Items2/iOS/ItemsViewController2.cs

 			{
 				_emptyViewFormsElement.Measure(frame.Width, frame.Height);
-				_emptyViewFormsElement.Arrange(frame.ToRectangle());
+				_emptyViewFormsElement.Arrange(new Rect(0, 0, frame.Width, frame.Height));


Could include a comment here?

// Arrange in the native container's local coordinate space (0,0). // The native container (_emptyUIView) is already positioned correctly by iOS, // so the MAUI element just needs to fill its container without additional offset.

@jsuarezruiz , I have added the comment

jsuarezruiz · 2025-10-06T07:25:56Z

src/Controls/src/Core/Handlers/Items2/iOS/ItemsViewController2.cs

@@ -601,7 +601,7 @@ virtual internal CGRect LayoutEmptyView()
 			if (_emptyViewFormsElement != null && ((IElementController)ItemsView).LogicalChildren.IndexOf(_emptyViewFormsElement) != -1)
 			{
 				_emptyViewFormsElement.Measure(frame.Width, frame.Height);


Should validate dimensions before use it?

@jsuarezruiz , Thanks for pointing out. I’ll add validation to ensure the width and height are greater than zero

src/Controls/tests/TestCases.HostApp/Issues/Issue31465.cs

jsuarezruiz · 2025-10-30T11:27:34Z

src/Controls/src/Core/Handlers/Items2/iOS/ItemsViewController2.cs

+					// Arrange in the native container's local coordinate space (0,0).
+					// The native container (_emptyUIView) is already positioned correctly by iOS,
+					// so the MAUI element just needs to fill its container without additional offset.
+					_emptyViewFormsElement.Arrange(new Rect(0, 0, frame.Width, frame.Height));


On iOS devices with notches/home indicators, safe area insets; is correct?

Yes @jsuarezruiz , I have ensured the fix with the latest changes on a notch screen device; it’s working fine.

Demo.mov

PureWeen · 2025-11-07T21:15:19Z

/rebase

sheiksyedm · 2025-11-11T10:35:31Z

/azp run

azure-pipelines · 2025-11-11T10:35:54Z

Azure Pipelines successfully started running 3 pipeline(s).

…space between Header and EmptyView text (dotnet#31840)  > [!NOTE] > Are you waiting for the changes in this PR to be merged? > It would be very helpful if you could [test the resulting artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from this PR and let us know in a comment if this change resolves your issue. Thank you!  ### Issue Details On iOS, a CollectionView with Header, EmptyView shows incorrect spacing when dragging down. The EmptyView shifts downward by double its intended Y offset leaving extra space between the header and empty view. ### Root Cause: The extra spacing happens because the header offset is applied twice. On iOS, the native container (_emptyUIView) is shifted by the ScrollView during header space calculations, and at the same time the MAUI element inside that container is also arranged with the same offset. As a result, the element ends up positioned once by its container and again by MAUI, which doubles the intended Y offset. ### Description of changes: Previously, the MAUI element was being arranged using absolute coordinates of the frame via _emptyViewFormsElement.Arrange(frame.ToRectangle()), which caused the header offset to be applied twice — once by the native container and once by the MAUI element. The fix changes the arrange call to _emptyViewFormsElement.Arrange(new Rect(0, 0, frame.Width, frame.Height)). This positions the MAUI element at the origin (0,0) within the native container while retaining its width and height. ### Issues Fixed  Fixes dotnet#31465  **Tested the behavior in the following platforms.** - [x] Android - [x] Windows - [x] iOS - [x] Mac | Before | After | |---------|--------| | **iOS**<br> <video src="https://github.com/user-attachments/assets/accebe0b-f969-43d0-8326-10de78ce2f74" width="300" height="600"> | **iOS**<br> <video src="https://github.com/user-attachments/assets/df9d7e7e-aefd-4132-95c3-5e93b815203b" width="300" height="600"> |

This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [Microsoft.Maui.Controls](https://github.com/dotnet/maui) | nuget | patch | `10.0.20` -> `10.0.30` | --- ### Release Notes <details> <summary>dotnet/maui (Microsoft.Maui.Controls)</summary> ### [`v10.0.30`](https://github.com/dotnet/maui/releases/tag/10.0.30): SR3 [Compare Source](dotnet/maui@10.0.20...10.0.30) #### What's Changed .NET MAUI 10.0.30 introduces significant improvements across all platforms with focus on quality, performance, and developer experience. This release includes 106 commits with various improvements, bug fixes, and enhancements. #### .NET 10 Features - \[net10.0] Merge main to net10.0 by [@PureWeen](https://github.com/PureWeen) in dotnet/maui#33323 #### CollectionView - Fixed the NRE in CarouselViewController on iOS 15.5 & 16.4 by [@Ahamed-Ali](https://github.com/Ahamed-Ali) in dotnet/maui#30838 <details> <summary>🔧 Fixes</summary> - [NRE in CarouselViewController on iOS 15.5 & 16.4](dotnet/maui#28557) </details> - \[iOS, macOS] Fixed CollectionView group header size changes with ItemSizingStrategy by [@NanthiniMahalingam](https://github.com/NanthiniMahalingam) in dotnet/maui#33161 <details> <summary>🔧 Fixes</summary> - [\[NET 10\] I6\_Grouping - Grouping_with_variable_sized_items changing the 'ItemSizingStrategy' also changes the header size.](dotnet/maui#33130) </details> - \[iOS]\[CV2] Fix page can be dragged down, and it would cause an extra space between Header and EmptyView text by [@devanathan-vaithiyanathan](https://github.com/devanathan-vaithiyanathan) in dotnet/maui#31840 <details> <summary>🔧 Fixes</summary> - [I8\_Header_and_Footer_Null - The page can be dragged down, and it would cause an extra space between Header and EmptyView text.](dotnet/maui#31465) </details> - \[iOS] Fixed the Items not displayed properly in CarouselView2 by [@Ahamed-Ali](https://github.com/Ahamed-Ali) in dotnet/maui#31336 <details> <summary>🔧 Fixes</summary> - [\[iOS\] Items are not updated properly in CarouselView2.](dotnet/maui#31148) </details> #### Docs - Add comprehensive README to Microsoft.Maui.Controls NuGet package by [@jfversluis](https://github.com/jfversluis) via [@Copilot](https://github.com/Copilot) in dotnet/maui#32835 <details> <summary>🔧 Fixes</summary> - [Improve the MAUI Nuget readme.md file](dotnet/maui#31969) </details> - Improve Controls Core API docs 2 by [@jfversluis](https://github.com/jfversluis) in https://github.com/dotnet...

dotnet-policy-service bot added community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration labels Oct 1, 2025

jsuarezruiz added the area-controls-collectionview CollectionView, CarouselView, IndicatorView label Oct 2, 2025

jsuarezruiz reviewed Oct 2, 2025

View reviewed changes

devanathan-vaithiyanathan marked this pull request as ready for review October 3, 2025 11:18

Copilot AI review requested due to automatic review settings October 3, 2025 11:18

Copilot AI reviewed Oct 3, 2025

View reviewed changes

jsuarezruiz suggested changes Oct 6, 2025

View reviewed changes

PureWeen added this to the .NET 10 SR1 milestone Oct 8, 2025

PureWeen added this to MAUI SDK Ongoing Oct 8, 2025

PureWeen moved this to Ready To Review in MAUI SDK Ongoing Oct 8, 2025

jsuarezruiz suggested changes Oct 30, 2025

View reviewed changes

github-project-automation bot moved this from Ready To Review to Changes Requested in MAUI SDK Ongoing Oct 30, 2025

PureWeen added the p/0 Current heighest priority issues that we are targeting for a release. label Nov 4, 2025

PureWeen linked an issue Nov 6, 2025 that may be closed by this pull request

I8_Header_and_Footer_Null - The page can be dragged down, and it would cause an extra space between Header and EmptyView text. #31465

Closed

PureWeen moved this from Changes Requested to Ready To Review in MAUI SDK Ongoing Nov 7, 2025

PureWeen changed the base branch from net10.0 to main November 7, 2025 21:15

devanathan-vaithiyanathan added 5 commits November 7, 2025 21:15

Update ItemsViewController2.cs

0ed81fa

Test case added

d8d3282

snapshots added

b3c4c07

Update ItemsViewController2.cs

cc17f44

Update ItemsViewController2.cs

1d95e88

github-actions bot force-pushed the Fix-31465 branch from f304f43 to 1d95e88 Compare November 7, 2025 21:16

github-actions bot locked and limited conversation to collaborators Jan 22, 2026

Conversation

devanathan-vaithiyanathan commented Oct 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue Details

Root Cause:

Description of changes:

Issues Fixed

Uh oh!

jsuarezruiz commented Oct 2, 2025

Uh oh!

azure-pipelines bot commented Oct 2, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Oct 3, 2025

Choose a reason for hiding this comment

Uh oh!

jsuarezruiz commented Oct 6, 2025

Uh oh!

azure-pipelines bot commented Oct 6, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

PureWeen commented Nov 7, 2025

Uh oh!

sheiksyedm commented Nov 11, 2025

Uh oh!

azure-pipelines bot commented Nov 11, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

devanathan-vaithiyanathan commented Oct 1, 2025 •

edited

Loading