[Testing] Feature matrix UITest Cases for ScrollView Control (#30188) by PureWeen · Pull Request #30991 · dotnet/maui

PureWeen · 2025-08-02T17:41:10Z

Description of Change

We checked in a set of ScrollView Matrix tests for NET9 that represented ScrollView behavior on NET9.

We've fixed a lot of issues with ScrollView in NET10 so these images need to be regenerated for the new more correct behavior.

This PR is a cherry-pick of those tests and regenerates all those images. There's one fix to iOS ScrollView that makes it match Android behavior now on NET10. In NET9 SCrollViews appeared to ignore the orientation property and were always scrollable in both directions even if the content > the size of the scrollview.

In NET10 this has been fixed so if you set orientation to Vertical it can only scroll vertical.

PureWeen · 2025-08-02T17:41:17Z

/azp run

azure-pipelines · 2025-08-02T17:41:33Z

Azure Pipelines successfully started running 3 pipeline(s).

Copilot

Pull Request Overview

This PR introduces a comprehensive UI test matrix for the ScrollView control in .NET MAUI to validate behavior against net10.0 changes. The implementation follows the standard feature matrix testing pattern used throughout the project.

Key Changes

Adds comprehensive ScrollView feature matrix tests covering various content types, orientations, and scroll behaviors
Implements platform-specific test exclusions for known issues on different platforms
Creates supporting infrastructure including ViewModel and UI pages for test configuration

Reviewed Changes

Copilot reviewed 9 out of 160 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
`src/Core/src/Platform/iOS/MauiScrollView.cs`	Implements content size clamping based on scroll orientation to prevent unwanted scrolling
`src/Core/src/Handlers/ContentView/ContentViewHandler.iOS.cs`	Adds ancestor measure invalidation after content view updates
`src/Controls/tests/TestCases.Shared.Tests/Tests/FeatureMatrix/ScrollViewFeatureTests.cs`	Comprehensive test suite with 1300+ lines covering ScrollView features across platforms
`src/Controls/tests/TestCases.HostApp/FeatureMatrix/ScrollView/ScrollViewViewModel.cs`	ViewModel for test configuration with scroll position tracking and content management
`src/Controls/tests/TestCases.HostApp/FeatureMatrix/ScrollView/ScrollViewOptionsPage.xaml.cs`	Options page code-behind handling test configuration UI interactions
`src/Controls/tests/TestCases.HostApp/FeatureMatrix/ScrollView/ScrollViewOptionsPage.xaml`	XAML configuration UI for selecting test options and content types
`src/Controls/tests/TestCases.HostApp/FeatureMatrix/ScrollView/ScrollViewControlPage.xaml.cs`	Main test page code-behind with scroll event handling and navigation
`src/Controls/tests/TestCases.HostApp/FeatureMatrix/ScrollView/ScrollViewControlPage.xaml`	Main test page XAML with ScrollView control and status displays
`src/Controls/tests/TestCases.HostApp/CoreViews/CorePageView.cs`	Registers ScrollView Feature Matrix in the gallery navigation

Comments suppressed due to low confidence (1)

Copilot · 2025-08-07T18:51:44Z

src/Controls/tests/TestCases.Shared.Tests/Tests/FeatureMatrix/ScrollViewFeatureTests.cs

+		VerifyScreenshot();
+	}
+
+	[Test , Order(43)]


There is an extra space before the comma in the Test attribute. It should be [Test, Order(43)] for consistency with other test declarations.

Copilot · 2025-08-07T18:51:44Z

src/Controls/tests/TestCases.Shared.Tests/Tests/FeatureMatrix/ScrollViewFeatureTests.cs

+		VerifyScreenshot();
+	}
+
+	[Test , Order(44)]


There is an extra space before the comma in the Test attribute. It should be [Test, Order(44)] for consistency with other test declarations.

Copilot · 2025-08-07T18:51:44Z

src/Controls/tests/TestCases.Shared.Tests/Tests/FeatureMatrix/ScrollViewFeatureTests.cs

+	}
+#endif
+
+    [Test]


Inconsistent indentation - this line uses spaces instead of tabs. Should use tabs for consistency with the rest of the file.

Copilot · 2025-08-07T18:51:44Z

src/Controls/tests/TestCases.HostApp/FeatureMatrix/ScrollView/ScrollViewOptionsPage.xaml.cs

+				{
+					Text = string.Join(Environment.NewLine, Enumerable.Range(1, 15).Select(i => $"Editor 1: Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed euismod, urna eu tincidunt consectetur, nisi nisl aliquam enim, eget facilisis enim nisl nec elit . Sed euismod, urna eu tincidunt consectetur, nisi nisl aliquam enim Eget facilisis enim nisl nec elit Vestibulum ante ipsum primis in faucibus orci luctus et ultrices posuere cubilia curae. Nullam ac erat at dui laoreet aliquet. Praesent euismod, justo at dictum facilisis, urna erat dictum enim.")),
+					HeightRequest = 250,
+					AutomationId="Editor1"


Missing space around the equals sign. Should be AutomationId = "Editor1" for consistency with other AutomationId assignments in the file.

Copilot · 2025-08-07T18:51:45Z

src/Controls/tests/TestCases.HostApp/FeatureMatrix/ScrollView/ScrollViewOptionsPage.xaml.cs

+				{
+					Text = string.Join(Environment.NewLine, Enumerable.Range(1, 15).Select(i => $"Editor 2: Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed euismod, urna eu tincidunt consectetur, nisi nisl aliquam enim, eget facilisis enim nisl nec elit . Sed euismod, urna eu tincidunt consectetur, nisi nisl aliquam enim Eget facilisis enim nisl nec elit Vestibulum ante ipsum primis in faucibus orci luctus et ultrices posuere cubilia curae. Nullam ac erat at dui laoreet aliquet. Praesent euismod, justo at dictum facilisis, urna erat dictum enim.")),
+					HeightRequest = 250,
+					AutomationId="Editor2"


Missing space around the equals sign. Should be AutomationId = "Editor2" for consistency with other AutomationId assignments in the file.

Copilot · 2025-08-07T18:51:45Z

src/Controls/tests/TestCases.HostApp/FeatureMatrix/ScrollView/ScrollViewOptionsPage.xaml.cs

+				{
+					Text = string.Join(Environment.NewLine, Enumerable.Range(1, 15).Select(i => $"Editor 3: Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed euismod, urna eu tincidunt consectetur, nisi nisl aliquam enim, eget facilisis enim nisl nec elit . Sed euismod, urna eu tincidunt consectetur, nisi nisl aliquam enim Eget facilisis enim nisl nec elit Vestibulum ante ipsum primis in faucibus orci luctus et ultrices posuere cubilia curae. Nullam ac erat at dui laoreet aliquet. Praesent euismod, justo at dictum facilisis, urna erat dictum enim.")),
+					HeightRequest = 250,
+					AutomationId="Editor3"


Missing space around the equals sign. Should be AutomationId = "Editor3" for consistency with other AutomationId assignments in the file.

Copilot · 2025-08-07T18:51:45Z

src/Controls/tests/TestCases.HostApp/FeatureMatrix/ScrollView/ScrollViewOptionsPage.xaml.cs

+				{
+					Text = string.Join(Environment.NewLine, Enumerable.Range(1, 15).Select(i => $"Editor 4: Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed euismod, urna eu tincidunt consectetur, nisi nisl aliquam enim, eget facilisis enim nisl nec elit . Sed euismod, urna eu tincidunt consectetur, nisi nisl aliquam enim Eget facilisis enim nisl nec elit Vestibulum ante ipsum primis in faucibus orci luctus et ultrices posuere cubilia curae. Nullam ac erat at dui laoreet aliquet. Praesent euismod, justo at dictum facilisis, urna erat dictum enim.")),
+					HeightRequest = 250,
+					AutomationId="Editor4"


Missing space around the equals sign. Should be AutomationId = "Editor4" for consistency with other AutomationId assignments in the file.

PureWeen · 2025-08-11T14:27:25Z

/rebase

* Added Feature Matrix for ScrollView * Updated changes * Changes updated * Modified the test cases * Modified UI * Added snapshots * Added images * Modified Test Case * Added images * Resaved images

PureWeen force-pushed the with_scrollview_tests branch from 0786bc0 to 99d384d Compare August 7, 2025 18:17

PureWeen marked this pull request as ready for review August 7, 2025 18:51

Copilot AI review requested due to automatic review settings August 7, 2025 18:51

PureWeen requested a review from a team as a code owner August 7, 2025 18:51

PureWeen requested review from StephaneDelcroix and mattleibow August 7, 2025 18:51

Copilot AI reviewed Aug 7, 2025

View reviewed changes

PureWeen assigned rmarinho Aug 11, 2025

NafeelaNazhir and others added 7 commits August 11, 2025 14:28

[Testing] Feature matrix UITest Cases for ScrollView Control (#30188)

e8270bd

* Added Feature Matrix for ScrollView * Updated changes * Changes updated * Modified the test cases * Modified UI * Added snapshots * Added images * Modified Test Case * Added images * Resaved images

android snap added (#31028)

c9f67f0

- fix iOS so that it clamps down on contentSize based on the orientation

716b511

- fix contentview invalidation so it propagates up the tree correctly

612225c

iOS snapshot added (#31080)

5ac62c5

- fix screenshots for fixed scrollview behavior

ecd7188

- slight adjustment of scrollview from android changes

86565e3

github-actions bot force-pushed the with_scrollview_tests branch from 9e0b440 to 86565e3 Compare August 11, 2025 14:28

rmarinho approved these changes Aug 13, 2025

View reviewed changes

rmarinho merged commit 2b13a90 into net10.0 Aug 13, 2025
127 of 129 checks passed

rmarinho deleted the with_scrollview_tests branch August 13, 2025 15:25

github-actions bot locked and limited conversation to collaborators Sep 13, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Testing] Feature matrix UITest Cases for ScrollView Control (#30188)#30991

[Testing] Feature matrix UITest Cases for ScrollView Control (#30188)#30991
rmarinho merged 7 commits intonet10.0from
with_scrollview_tests

PureWeen commented Aug 2, 2025 •

edited

Loading

Uh oh!

PureWeen commented Aug 2, 2025

Uh oh!

azure-pipelines bot commented Aug 2, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Aug 7, 2025

Uh oh!

Copilot AI Aug 7, 2025

Uh oh!

Copilot AI Aug 7, 2025

Uh oh!

Copilot AI Aug 7, 2025

Uh oh!

Copilot AI Aug 7, 2025

Uh oh!

Copilot AI Aug 7, 2025

Uh oh!

Copilot AI Aug 7, 2025

Uh oh!

PureWeen commented Aug 11, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

+              	}
+              #endif
+                  [Test]

Conversation

PureWeen commented Aug 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description of Change

Uh oh!

PureWeen commented Aug 2, 2025

Uh oh!

azure-pipelines bot commented Aug 2, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Key Changes

Reviewed Changes

Uh oh!

Copilot AI Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

PureWeen commented Aug 11, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

PureWeen commented Aug 2, 2025 •

edited

Loading