[Windows] FlyoutPage: Fix CollapsedPaneWidth mapping and updates by devanathan-vaithiyanathan · Pull Request #33786 · dotnet/maui

devanathan-vaithiyanathan · 2026-01-30T07:55:03Z

Issue Details

The CollapsedPaneWidth value was not being properly applied to the native NavigationView.

Description of Change

On Windows, FlyoutPage.CollapsedPaneWidth was not being propagated to the native RootNavigationView, so changing the attached property had no effect on CompactPaneLength.

This PR wires PlatformConfiguration.WindowsSpecific.FlyoutPage.CollapsedPaneWidthProperty into the FlyoutPage mapper and applies the value to RootNavigationView.CompactPaneLength. It also adds a property-changed callback so runtime changes call Handler.UpdateValue(...) and update the native control after the handler is created.

A Windows UI test was added to cover both the initial partial-collapse setup and a runtime width change.

Note : FlyoutPage CollapsableStyle issue was resolved in PR #29927

Issues Fixed

Fixes #33785

Tested the behavior in the following platforms.

Android
Windows
iOS
Mac

Before	After
Windows Before.mp4	Windows After.mp4

sheiksyedm · 2026-01-30T13:31:41Z

/azp run maui-pr-uitests

azure-pipelines · 2026-01-30T13:31:51Z

Azure Pipelines successfully started running 1 pipeline(s).

Copilot

Pull request overview

This PR fixes the FlyoutPage CollapsedPaneWidth property on Windows by implementing proper mapping from the MAUI property to the native NavigationView.CompactPaneLength property. The fix enables both initial and dynamic configuration of the collapsed pane width when using CollapseStyle.Partial.

Changes:

Added property change callback to trigger handler updates when CollapsedPaneWidth changes
Registered mapper for CollapsedPaneWidth property in FlyoutPage handler
Implemented mapper to apply CollapsedPaneWidth to native NavigationView.CompactPaneLength
Added UI test to verify CollapsedPaneWidth works both initially and when changed dynamically

Reviewed changes

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

File	Description
src/Controls/src/Core/PlatformConfiguration/WindowsSpecific/FlyoutPage.cs	Added property changed callback (OnCollapsedPaneWidthChanged) to trigger handler updates when CollapsedPaneWidth property changes
src/Controls/src/Core/FlyoutPage/FlyoutPage.Mapper.cs	Registered mapper for CollapsedPaneWidthProperty and implemented MapCollapsedPaneWidth to apply the value to NavigationView.CompactPaneLength
src/Controls/tests/TestCases.HostApp/Issues/Issue33785.cs	Created UI test page that demonstrates CollapsedPaneWidth functionality with initial value (50) and dynamic change (100)
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue33785.cs	Implemented automated UI test that verifies CollapsedPaneWidth changes are applied correctly using screenshot verification

Copilot · 2026-01-30T13:41:13Z

src/Controls/src/Core/FlyoutPage/FlyoutPage.Mapper.cs

+			if (view is BindableObject bindable && handler.PlatformView is Microsoft.Maui.Platform.RootNavigationView navigationView)
+			{
+				var collapsedPaneWidth = PlatformConfiguration.WindowsSpecific.FlyoutPage.GetCollapsedPaneWidth(bindable);
+				if (collapsedPaneWidth > 0)


The validation condition if (collapsedPaneWidth > 0) is inconsistent with the property's validation which allows values >= 0. A value of 0 is valid according to the BindableProperty definition (line 70 in FlyoutPage.cs), but this mapper would skip applying it. Consider changing the condition to >= 0 or removing it entirely since the property already validates non-negative values. The default value is 48, so the typical case of 0 would represent "use default" behavior, which should probably still be applied to the platform control.

Suggested change

if (collapsedPaneWidth > 0)

if (collapsedPaneWidth >= 0)

MauiBot · 2026-03-18T23:40:09Z

🤖 AI Summary

📊 Expand Full Review — 43386ab · mauibot concern addressed

🔍 Pre-Flight — Context & Validation

Issue: #33785 - FlyoutPage CollapsedPaneWidth is not working
PR: #33786 - [Windows] FlyoutPage: Fix CollapsedPaneWidth mapping and updates
Platforms Affected: Windows only
Files Changed: 2 implementation, 2 test

Key Findings

FlyoutPage.CollapsedPaneWidth (Windows-specific attached property, default 48) was not wired into the MAUI handler mapper, so it had no effect on RootNavigationView.CompactPaneLength. The old Compatibility layer (FlyoutPageControl.cs) did apply it, but the new handler system was missing the mapping.
PR adds a #if WINDOWS mapper entry in FlyoutPage.Mapper.cs that reads GetCollapsedPaneWidth() and sets CompactPaneLength on the native RootNavigationView.
PR adds an OnCollapsedPaneWidthChanged propertyChanged callback on CollapsedPaneWidthProperty so runtime changes also call handler.UpdateValue(nameof(CollapsedPaneWidthProperty)) — this ensures the mapper fires on subsequent property changes.
The nameof strings used in the callback ("CollapsedPaneWidthProperty") and in the mapper registration (nameof(PlatformConfiguration.WindowsSpecific.FlyoutPage.CollapsedPaneWidthProperty)) both resolve to "CollapsedPaneWidthProperty" — they are consistent.
Gate FAILED (Windows): The test calls VerifyScreenshot() with no committed baseline image. VerifyScreenshot() compares to a file that doesn't exist in the repo, so the test fails regardless of whether the functional fix is correct.
Prior agent review (March 2026): Previous review ran on MacCatalyst (gate skipped as platform mismatch). 6 try-fix attempts were all BLOCKED by the same missing baseline screenshot issue. All models agreed the missing baseline is the real blocker, not the implementation.
Inline review comment (now outdated): Previous bot review flagged if (collapsedPaneWidth > 0) — this condition no longer exists in the current code; the mapper now unconditionally sets CompactPaneLength = collapsedPaneWidth.
CollapseStyle interaction: CollapseStyle (Partial vs Full) was already fixed in PR [Windows] FlyoutPage: update CollapseStyle at runtime #29927. The current PR only needs to ensure CompactPaneLength gets the right value when in LeftCompact mode.
Minor: Test files (Issue33785.cs in SharedTests) are missing a newline at end of file.
FlyoutLayoutBehavior = Popover is set in the test page — but CollapseStyle.Partial is also set. On Windows, Popover means LeftMinimal / Overlay, which may cause CompactPaneLength to have no visual effect since there's no compact pane displayed. This is a potential test design issue.

Fix Candidates

#	Source	Approach	Test Result	Files Changed	Notes
PR	PR #33786	Add dedicated Windows `CollapsedPaneWidth` mapper + `propertyChanged` callback	❌ FAILED (Gate - missing baseline)	4 files	Core logic appears correct; test fails due to missing `VerifyScreenshot()` baseline

🔧 Fix — Analysis & Comparison

Fix Candidates

#	Source	Approach	Test Result	Files Changed	Notes
1	try-fix	Integrate CollapseStyle + CompactPaneLength into existing FlyoutBehavior mapper; also switch PaneDisplayMode to LeftCompact when CollapseStyle.Partial; replace VerifyScreenshot() with label assertion	✅ PASS	3 files	Reveals PR's critical bug: PR sets CompactPaneLength but stays in LeftMinimal mode where it has no effect
2	try-fix	Interface-based approach (IPartialCollapseView): FlyoutPage implements interface in Controls; Core FlyoutViewHandler.Windows.cs checks interface in MapFlyoutBehavior and applies LeftCompact + CompactPaneLength; label assertion test	✅ PASS	5 files	Architecturally cleaner Core/Controls boundary; also fixes PaneDisplayMode gap
3	try-fix	(pending)	⏳	-	-
4	try-fix	(pending)	⏳	-	-
PR	PR #33786	Dedicated CollapsedPaneWidth mapper + propertyChanged callback; VerifyScreenshot() test	❌ FAILED (Gate)	4 files	Missing baseline snapshot; also misses PaneDisplayMode=LeftCompact needed for CompactPaneLength to work

Cross-Pollination

Model	Round	New Ideas?	Details
(pending round 2)	-	-	-

Exhausted: No
Selected Fix: Pending all attempts

MauiBot · 2026-03-18T23:40:10Z

🤖 AI Summary

📊 Expand Full Review — 43386ab · mauibot concern addressed

🔍 Pre-Flight — Context & Validation

Issue: #33785 - FlyoutPage CollapsedPaneWidth is not working
PR: #33786 - [Windows] FlyoutPage: Fix CollapsedPaneWidth mapping and updates
Platforms Affected: Windows only
Files Changed: 2 implementation, 2 test

Key Findings

FlyoutPage.CollapsedPaneWidth (Windows-specific attached property, default 48) was not wired into the MAUI handler mapper. Setting it had no effect on RootNavigationView.CompactPaneLength. The old Compatibility layer applied it, but the new handler system was missing the mapping entirely.
PR adds a #if WINDOWS mapper entry in FlyoutPage.Mapper.cs calling ReplaceMapping with MapCollapsedPaneWidth, which reads GetCollapsedPaneWidth() and sets navigationView.CompactPaneLength.
PR adds an OnCollapsedPaneWidthChanged propertyChanged callback on CollapsedPaneWidthProperty so runtime changes call handler.UpdateValue(nameof(CollapsedPaneWidthProperty)).
Gate FAILED: VerifyScreenshot() in the test fails because no baseline PNG is committed to the repo. Error: "Baseline snapshot not yet created: ...snapshots/windows/VerifyFlyoutPageCollapsedPaneWidth...". Both "Without Fix" and "With Fix" runs fail for this same reason — the gate can't differentiate functional behavior.
PR author explained: they need PR [Windows] FlyoutPage: update CollapseStyle at runtime #29927 (CollapseStyle fix) merged first before adding the baseline screenshot.
Key consistency check: nameof(CollapsedPaneWidthProperty) in OnCollapsedPaneWidthChanged → "CollapsedPaneWidthProperty", and nameof(PlatformConfiguration.WindowsSpecific.FlyoutPage.CollapsedPaneWidthProperty) in the mapper registration → "CollapsedPaneWidthProperty". Both resolve to the same string — the mapping is consistent.
Prior Copilot inline review: Flagged if (collapsedPaneWidth > 0) guard that would skip applying value 0. This is now marked outdated/resolved — the current mapper has no such guard.
CollapseStyleProperty has no property-changed handler to trigger handler updates — this means SetCollapseStyle() at runtime wouldn't re-map (out of scope for this PR, but related).

Fix Candidates

#	Source	Approach	Test Result	Files Changed	Notes
PR	PR #33786	Add mapper + propertyChanged callback for CollapsedPaneWidthProperty	❌ FAILED (no baseline snapshot)	`FlyoutPage.Mapper.cs`, `WindowsSpecific/FlyoutPage.cs`	Functionally likely correct; test design requires baseline screenshot

🔧 Fix — Analysis & Comparison

Fix Candidates

#	Source	Approach	Test Result	Files Changed	Notes
PR	PR #33786	`ReplaceMapping` → new `MapCollapsedPaneWidth` + propertyChanged → `UpdateValue(CollapsedPaneWidthProperty)`	❌ FAIL (no baseline snapshot)	`FlyoutPage.Mapper.cs`, `WindowsSpecific/FlyoutPage.cs`	Functionally likely correct; test needs baseline PNG
1	try-fix (claude-opus-4.6)	Piggyback `MapFlyoutLayoutBehavior` + `UpdateCollapsedPaneWidth` in `NavigationViewExtensions.cs`; propertyChanged → `UpdateValue(FlyoutLayoutBehavior)`; label text assertion in test	✅ PASS	`FlyoutPage.Mapper.cs`, `WindowsSpecific/FlyoutPage.cs`, `NavigationViewExtensions.cs`, `Issue33785.cs` (test)	Reuses existing mapper pattern; no new mapper key
2	try-fix (claude-sonnet-4.6)	`OnHandlerChanged` override + direct `PlatformView` cast in propertyChanged (bypasses mapper pipeline)	✅ PASS	`FlyoutPage.Mapper.cs`, `WindowsSpecific/FlyoutPage.cs`, `Issue33785.cs` (test), `PublicAPI.Unshipped.txt`	More invasive; expands public API surface
3	try-fix (gpt-5.3-codex)	Handler-level via `FlyoutViewHandler.Windows.MapFlyoutBehavior` + reflection; propertyChanged → `UpdateValue(IFlyoutView.FlyoutBehavior)`	✅ PASS	`FlyoutViewHandler.Windows.cs`, `WindowsSpecific/FlyoutPage.cs`, `Issue33785.cs` (test)	Uses reflection (fragile); touches Core handler
4	try-fix (gpt-5.4)	Only `NavigationViewExtensions.UpdateFlyoutBehavior` — no propertyChanged at all	❌ FAIL	`NavigationViewExtensions.cs`, `Issue33785.cs` (test)	Doesn't handle runtime property changes

Cross-Pollination

Model	Round	New Ideas?	Details
claude-opus-4.6	2	No	"Solution space is effectively covered — all meaningful combinations of location × trigger are represented"

Exhausted: Yes
Selected Fix: PR's fix approach (implementation) + Attempt 1's test fix — Reason: The PR's mapper implementation is the cleanest pattern (mirrors existing iOS-specific mappers exactly). The only blocker is the test using VerifyScreenshot() without a committed baseline. Replacing VerifyScreenshot() with a label text assertion (as done in all three passing attempts) resolves the gate failure without changing the clean PR implementation.

📋 Report — Final Recommendation

⚠️ Final Recommendation: REQUEST CHANGES

Phase Status

Phase	Status	Notes
Pre-Flight	✅ COMPLETE	Issue #33785 · Windows FlyoutPage CollapsedPaneWidth not applied
Gate	❌ FAILED	Windows — both runs fail: "Baseline snapshot not yet created"
Try-Fix	✅ COMPLETE	4 attempts; 3 passing (Attempts 1–3), 1 failing (Attempt 4)
Report	✅ COMPLETE

Summary

PR #33786 fixes a real bug: FlyoutPage.CollapsedPaneWidth (a Windows-specific attached property) was never wired into the handler mapper, so it had no effect on RootNavigationView.CompactPaneLength. The implementation approach — a dedicated #if WINDOWS mapper entry in FlyoutPage.Mapper.cs plus a propertyChanged callback — is correct and follows the established pattern (mirroring the iOS-specific mappers in the same file).

The Gate failed for one reason only: the test calls VerifyScreenshot() but no baseline PNG has been committed to the repository. This makes the test fail unconditionally regardless of whether the fix works.

Root Cause

Bug: CollapsedPaneWidthProperty was defined and validated but never registered in the FlyoutPage mapper. No path existed to apply it to RootNavigationView.CompactPaneLength — either at initial handler connection or on runtime changes.

Gate failure: VerifyScreenshot() compares against a committed PNG at snapshots/windows/VerifyFlyoutPageCollapsedPaneWidth.png. No such file exists. The PR author acknowledged this and said they'd add the image once PR #29927 (CollapseStyle) is merged.

Fix Quality

Implementation — ✅ Correct. The PR's mapper approach is clean and idiomatic:

ReplaceMapping in FlyoutPage.Mapper.cs follows the exact same pattern as the existing iOS-specific mappers in that file
The nameof keys are consistent: both nameof(CollapsedPaneWidthProperty) and nameof(PlatformConfiguration.WindowsSpecific.FlyoutPage.CollapsedPaneWidthProperty) resolve to "CollapsedPaneWidthProperty"
No guard condition (the previously-flagged > 0 issue) — the Copilot inline review was addressed

Test — ❌ Needs fixing. Two valid paths:

Commit a baseline screenshot — run the test, capture the output PNG, commit it to src/Controls/tests/TestCases.Shared.Tests/snapshots/windows/. This is the standard approach for screenshot-based tests.
Use structural assertion instead — replace VerifyScreenshot() with a label text assertion. All three passing try-fix attempts used this approach successfully:
```
App.TapFlyoutPageIcon();
App.Tap("FlyoutItem");
App.TapFlyoutPageIcon();
var labelText = App.FindElement("CollapsedPaneLabel").GetText();
Assert.That(labelText, Is.EqualTo("CollapsedPaneWidth set to 100"));
```
This verifies the runtime property change fired and applied correctly without needing a visual baseline.

Recommendation: Option 2 (structural assertion) is strongly preferred here because:

The test is verifying functional behavior (property change applied), not visual appearance
Visual layout of the flyout pane width varies with machine DPI/resolution — screenshot tests for pane widths are fragile
No dependency on PR [Windows] FlyoutPage: update CollapseStyle at runtime #29927 being merged first

Required Changes

Test fix (blocking): Replace VerifyScreenshot() in Issue33785.cs with a label text assertion as shown above
Minor: PlatformAffected.UWP in [Issue(...)] attribute on the HostApp page — PlatformAffected.All or PlatformAffected.UWP are both acceptable; UWP is the conventional choice for Windows-only issues in this codebase

Selected Fix: Alternative fix (Attempt 1) preferred for test; PR implementation is correct

github-actions · 2026-03-19T07:08:43Z

🚀 Dogfood this PR with:

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

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

Or

Run remotely in PowerShell:

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

kubaflo · 2026-03-19T12:56:25Z

/azp run maui-pr-uitests

kubaflo · 2026-03-19T12:56:38Z

Could you please add the Windows snapshot? :)

azure-pipelines · 2026-03-19T12:56:39Z

Azure Pipelines successfully started running 1 pipeline(s).

MauiBot · 2026-03-28T18:52:59Z

🚦 Gate - Test Before and After Fix

📊 Expand Full Gate — 43386ab · mauibot concern addressed

Gate Result: ❌ FAILED

Platform: WINDOWS · Base: main · Merge base: 794a9fa6

Test	Without Fix (expect FAIL)	With Fix (expect PASS)
🖥️ Issue33785 `Issue33785`	✅ FAIL — 563s	❌ FAIL — 478s

🔴 Without fix — 🖥️ Issue33785: FAIL ✅ · 563s

  Determining projects to restore...
  Restored D:\a\1\s\src\Graphics\src\Graphics\Graphics.csproj (in 27.94 sec).
  Restored D:\a\1\s\src\Graphics\src\Graphics.Win2D\Graphics.Win2D.csproj (in 27.99 sec).
  Restored D:\a\1\s\src\Essentials\src\Essentials.csproj (in 10.38 sec).
  Restored D:\a\1\s\src\Core\src\Core.csproj (in 15.66 sec).
  Restored D:\a\1\s\src\Core\maps\src\Maps.csproj (in 11.14 sec).
  Restored D:\a\1\s\src\Controls\tests\TestCases.HostApp\Controls.TestCases.HostApp.csproj (in 5.84 sec).
  Restored D:\a\1\s\src\Controls\src\Core\Controls.Core.csproj (in 18 ms).
  Restored D:\a\1\s\src\Controls\src\Xaml\Controls.Xaml.csproj (in 33 ms).
  Restored D:\a\1\s\src\Controls\Maps\src\Controls.Maps.csproj (in 12 ms).
  Restored D:\a\1\s\src\Controls\Foldable\src\Controls.Foldable.csproj (in 9 ms).
  Restored D:\a\1\s\src\BlazorWebView\src\Maui\Microsoft.AspNetCore.Components.WebView.Maui.csproj (in 26 ms).
  3 of 14 projects are up-to-date for restore.
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Graphics -> D:\a\1\s\artifacts\bin\Graphics\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Graphics.Win2D -> D:\a\1\s\artifacts\bin\Graphics.Win2D\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Graphics.Win2D.WinUI.Desktop.dll
  Essentials -> D:\a\1\s\artifacts\bin\Essentials\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Core -> D:\a\1\s\artifacts\bin\Core\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Maps -> D:\a\1\s\artifacts\bin\Maps\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Maps.dll
  Controls.BindingSourceGen -> D:\a\1\s\artifacts\bin\Controls.BindingSourceGen\Debug\netstandard2.0\Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Controls.Core -> D:\a\1\s\artifacts\bin\Controls.Core\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Controls.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Controls.Xaml -> D:\a\1\s\artifacts\bin\Controls.Xaml\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Controls.Xaml.dll
  Controls.Foldable -> D:\a\1\s\artifacts\bin\Controls.Foldable\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Controls.Foldable.dll
  Microsoft.AspNetCore.Components.WebView.Maui -> D:\a\1\s\artifacts\bin\Microsoft.AspNetCore.Components.WebView.Maui\Debug\net10.0-windows10.0.19041.0\Microsoft.AspNetCore.Components.WebView.Maui.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Controls.Maps -> D:\a\1\s\artifacts\bin\Controls.Maps\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Controls.Maps.dll
  Controls.TestCases.HostApp -> D:\a\1\s\artifacts\bin\Controls.TestCases.HostApp\Debug\net10.0-windows10.0.19041.0\win-x64\Controls.TestCases.HostApp.dll

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

Time Elapsed 00:05:48.25
  Determining projects to restore...
  Restored D:\a\1\s\src\TestUtils\src\VisualTestUtils\VisualTestUtils.csproj (in 784 ms).
  Restored D:\a\1\s\src\TestUtils\src\UITest.NUnit\UITest.NUnit.csproj (in 1.45 sec).
  Restored D:\a\1\s\src\TestUtils\src\UITest.Core\UITest.Core.csproj (in 3 ms).
  Restored D:\a\1\s\src\TestUtils\src\UITest.Appium\UITest.Appium.csproj (in 997 ms).
  Restored D:\a\1\s\src\TestUtils\src\UITest.Analyzers\UITest.Analyzers.csproj (in 9.08 sec).
  Restored D:\a\1\s\src\TestUtils\src\VisualTestUtils.MagickNet\VisualTestUtils.MagickNet.csproj (in 20.08 sec).
  Restored D:\a\1\s\src\Controls\tests\CustomAttributes\Controls.CustomAttributes.csproj (in 4 ms).
  Restored D:\a\1\s\src\Controls\tests\TestCases.WinUI.Tests\Controls.TestCases.WinUI.Tests.csproj (in 7.67 sec).
  7 of 15 projects are up-to-date for restore.
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Graphics -> D:\a\1\s\artifacts\bin\Graphics\Debug\net10.0\Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Essentials -> D:\a\1\s\artifacts\bin\Essentials\Debug\net10.0\Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Controls.CustomAttributes -> D:\a\1\s\artifacts\bin\Controls.CustomAttributes\Debug\net10.0\Controls.CustomAttributes.dll
  Core -> D:\a\1\s\artifacts\bin\Core\Debug\net10.0\Microsoft.Maui.dll
  Controls.Core.Design -> D:\a\1\s\artifacts\bin\Controls.Core.Design\Debug\net472\Microsoft.Maui.Controls.DesignTools.dll
  Controls.BindingSourceGen -> D:\a\1\s\artifacts\bin\Controls.BindingSourceGen\Debug\netstandard2.0\Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Controls.Core -> D:\a\1\s\artifacts\bin\Controls.Core\Debug\net10.0\Microsoft.Maui.Controls.dll
  UITest.Core -> D:\a\1\s\artifacts\bin\UITest.Core\Debug\net10.0\UITest.Core.dll
  UITest.Appium -> D:\a\1\s\artifacts\bin\UITest.Appium\Debug\net10.0\UITest.Appium.dll
  UITest.NUnit -> D:\a\1\s\artifacts\bin\UITest.NUnit\Debug\net10.0\UITest.NUnit.dll
  VisualTestUtils -> D:\a\1\s\artifacts\bin\VisualTestUtils\Debug\netstandard2.0\VisualTestUtils.dll
  VisualTestUtils.MagickNet -> D:\a\1\s\artifacts\bin\VisualTestUtils.MagickNet\Debug\netstandard2.0\VisualTestUtils.MagickNet.dll
  UITest.Analyzers -> D:\a\1\s\artifacts\bin\UITest.Analyzers\Debug\netstandard2.0\UITest.Analyzers.dll
  Controls.TestCases.WinUI.Tests -> D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\Controls.TestCases.WinUI.Tests.dll
Test run for D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\Controls.TestCases.WinUI.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (x64)

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\Controls.TestCases.WinUI.Tests.dll
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\Controls.TestCases.WinUI.Tests.dll
   NUnit3TestExecutor discovered 1 of 1 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 4/2/2026 10:43:42 PM FixtureSetup for Issue33785(Windows)
>>>>> 4/2/2026 10:43:51 PM VerifyFlyoutPageCollapsedPaneWidth Start
>>>>> 4/2/2026 10:43:55 PM VerifyFlyoutPageCollapsedPaneWidth Stop
>>>>> 4/2/2026 10:43:55 PM Log types: 
  Failed VerifyFlyoutPageCollapsedPaneWidth [4 s]
  Error Message:
   VisualTestUtils.VisualTestFailedException : 
Baseline snapshot not yet created: D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\snapshots\windows\VerifyFlyoutPageCollapsedPaneWidth.png
Ensure new snapshot is correct:    D:\a\1\a\Controls.TestCases.Shared.Tests\snapshots-diff\windows\VerifyFlyoutPageCollapsedPaneWidth.png
  and if it is, push a change to add it to the 'snapshots' directory.
See test attachment or download the build artifacts to get the new snapshot file.

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

  Stack Trace:
     at VisualTestUtils.VisualRegressionTester.Fail(String message) in /_/src/TestUtils/src/VisualTestUtils/VisualRegressionTester.cs:line 162
   at VisualTestUtils.VisualRegressionTester.VerifyMatchesSnapshot(String name, ImageSnapshot actualImage, String environmentName, ITestContext testContext) in /_/src/TestUtils/src/VisualTestUtils/VisualRegressionTester.cs:line 84
   at Microsoft.Maui.TestCases.Tests.UITest.<VerifyScreenshot>g__Verify|13_0(String name, <>c__DisplayClass13_0&) in /_/src/Controls/tests/TestCases.Shared.Tests/UITest.cs:line 477
   at Microsoft.Maui.TestCases.Tests.UITest.VerifyScreenshot(String name, Nullable`1 retryDelay, Nullable`1 retryTimeout, Int32 cropLeft, Int32 cropRight, Int32 cropTop, Int32 cropBottom, Double tolerance, Boolean includeTitleBar) in /_/src/Controls/tests/TestCases.Shared.Tests/UITest.cs:line 309
   at Microsoft.Maui.TestCases.Tests.Issues.Issue33785.VerifyFlyoutPageCollapsedPaneWidth() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue33785.cs:line 20
   at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
   at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)

NUnit Adapter 4.5.0.0: Test execution complete
[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.09]   Discovering: Controls.TestCases.WinUI.Tests
[xUnit.net 00:00:00.35]   Discovered:  Controls.TestCases.WinUI.Tests

Total tests: 1
     Failed: 1
Test Run Failed.
 Total time: 31.9082 Seconds

🟢 With fix — 🖥️ Issue33785: FAIL ❌ · 478s

  Determining projects to restore...
  All projects are up-to-date for restore.
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Graphics -> D:\a\1\s\artifacts\bin\Graphics\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Essentials -> D:\a\1\s\artifacts\bin\Essentials\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Essentials.dll
  Graphics.Win2D -> D:\a\1\s\artifacts\bin\Graphics.Win2D\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Graphics.Win2D.WinUI.Desktop.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Core -> D:\a\1\s\artifacts\bin\Core\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.dll
  Controls.BindingSourceGen -> D:\a\1\s\artifacts\bin\Controls.BindingSourceGen\Debug\netstandard2.0\Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Maps -> D:\a\1\s\artifacts\bin\Maps\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Maps.dll
  Controls.Core -> D:\a\1\s\artifacts\bin\Controls.Core\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Controls.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Controls.Xaml -> D:\a\1\s\artifacts\bin\Controls.Xaml\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Controls.Xaml.dll
  Microsoft.AspNetCore.Components.WebView.Maui -> D:\a\1\s\artifacts\bin\Microsoft.AspNetCore.Components.WebView.Maui\Debug\net10.0-windows10.0.19041.0\Microsoft.AspNetCore.Components.WebView.Maui.dll
  Controls.Foldable -> D:\a\1\s\artifacts\bin\Controls.Foldable\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Controls.Foldable.dll
  Controls.Maps -> D:\a\1\s\artifacts\bin\Controls.Maps\Debug\net10.0-windows10.0.19041.0\Microsoft.Maui.Controls.Maps.dll
  Controls.TestCases.HostApp -> D:\a\1\s\artifacts\bin\Controls.TestCases.HostApp\Debug\net10.0-windows10.0.19041.0\win-x64\Controls.TestCases.HostApp.dll

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

Time Elapsed 00:05:46.00
  Determining projects to restore...
  All projects are up-to-date for restore.
  Controls.CustomAttributes -> D:\a\1\s\artifacts\bin\Controls.CustomAttributes\Debug\net10.0\Controls.CustomAttributes.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Graphics -> D:\a\1\s\artifacts\bin\Graphics\Debug\net10.0\Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Essentials -> D:\a\1\s\artifacts\bin\Essentials\Debug\net10.0\Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Core -> D:\a\1\s\artifacts\bin\Core\Debug\net10.0\Microsoft.Maui.dll
  Controls.Core.Design -> D:\a\1\s\artifacts\bin\Controls.Core.Design\Debug\net472\Microsoft.Maui.Controls.DesignTools.dll
  Controls.BindingSourceGen -> D:\a\1\s\artifacts\bin\Controls.BindingSourceGen\Debug\netstandard2.0\Microsoft.Maui.Controls.BindingSourceGen.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13730048
  Controls.Core -> D:\a\1\s\artifacts\bin\Controls.Core\Debug\net10.0\Microsoft.Maui.Controls.dll
  UITest.Core -> D:\a\1\s\artifacts\bin\UITest.Core\Debug\net10.0\UITest.Core.dll
  UITest.Appium -> D:\a\1\s\artifacts\bin\UITest.Appium\Debug\net10.0\UITest.Appium.dll
  UITest.NUnit -> D:\a\1\s\artifacts\bin\UITest.NUnit\Debug\net10.0\UITest.NUnit.dll
  VisualTestUtils -> D:\a\1\s\artifacts\bin\VisualTestUtils\Debug\netstandard2.0\VisualTestUtils.dll
  VisualTestUtils.MagickNet -> D:\a\1\s\artifacts\bin\VisualTestUtils.MagickNet\Debug\netstandard2.0\VisualTestUtils.MagickNet.dll
  UITest.Analyzers -> D:\a\1\s\artifacts\bin\UITest.Analyzers\Debug\netstandard2.0\UITest.Analyzers.dll
  Controls.TestCases.WinUI.Tests -> D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\Controls.TestCases.WinUI.Tests.dll
Test run for D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\Controls.TestCases.WinUI.Tests.dll (.NETCoreApp,Version=v10.0)
VSTest version 18.0.1 (x64)

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\Controls.TestCases.WinUI.Tests.dll
NUnit Adapter 4.5.0.0: Test execution started
Running selected tests in D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\Controls.TestCases.WinUI.Tests.dll
   NUnit3TestExecutor discovered 1 of 1 NUnit test cases using Current Discovery mode, Non-Explicit run
>>>>> 4/2/2026 10:51:41 PM FixtureSetup for Issue33785(Windows)
>>>>> 4/2/2026 10:51:49 PM VerifyFlyoutPageCollapsedPaneWidth Start
>>>>> 4/2/2026 10:51:53 PM VerifyFlyoutPageCollapsedPaneWidth Stop
>>>>> 4/2/2026 10:51:53 PM Log types: 
  Failed VerifyFlyoutPageCollapsedPaneWidth [4 s]
  Error Message:
   VisualTestUtils.VisualTestFailedException : 
Baseline snapshot not yet created: D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\snapshots\windows\VerifyFlyoutPageCollapsedPaneWidth.png
Ensure new snapshot is correct:    D:\a\1\a\Controls.TestCases.Shared.Tests\snapshots-diff\windows\VerifyFlyoutPageCollapsedPaneWidth.png
  and if it is, push a change to add it to the 'snapshots' directory.
See test attachment or download the build artifacts to get the new snapshot file.

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

  Stack Trace:
     at VisualTestUtils.VisualRegressionTester.Fail(String message) in /_/src/TestUtils/src/VisualTestUtils/VisualRegressionTester.cs:line 162
   at VisualTestUtils.VisualRegressionTester.VerifyMatchesSnapshot(String name, ImageSnapshot actualImage, String environmentName, ITestContext testContext) in /_/src/TestUtils/src/VisualTestUtils/VisualRegressionTester.cs:line 84
   at Microsoft.Maui.TestCases.Tests.UITest.<VerifyScreenshot>g__Verify|13_0(String name, <>c__DisplayClass13_0&) in /_/src/Controls/tests/TestCases.Shared.Tests/UITest.cs:line 477
   at Microsoft.Maui.TestCases.Tests.UITest.VerifyScreenshot(String name, Nullable`1 retryDelay, Nullable`1 retryTimeout, Int32 cropLeft, Int32 cropRight, Int32 cropTop, Int32 cropBottom, Double tolerance, Boolean includeTitleBar) in /_/src/Controls/tests/TestCases.Shared.Tests/UITest.cs:line 309
   at Microsoft.Maui.TestCases.Tests.Issues.Issue33785.VerifyFlyoutPageCollapsedPaneWidth() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue33785.cs:line 20
   at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
   at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)

NUnit Adapter 4.5.0.0: Test execution complete
[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.10]   Discovering: Controls.TestCases.WinUI.Tests
[xUnit.net 00:00:00.34]   Discovered:  Controls.TestCases.WinUI.Tests

Total tests: 1
     Failed: 1
Test Run Failed.
 Total time: 27.0842 Seconds

⚠️ Issues found

❌ Issue33785 FAILED with fix (should pass)
- VerifyFlyoutPageCollapsedPaneWidth [4 s]
- VisualTestUtils.VisualTestFailedException : Baseline snapshot not yet created: D:\a\1\s\artifacts\bin\Controls.TestCases.WinUI.Tests\Debug\net10.0\snapshots\windows\VerifyFlyoutPageCollapsedPaneWidt...

📁 Fix files reverted (3 files)

eng/pipelines/ci-copilot.yml
src/Controls/src/Core/FlyoutPage/FlyoutPage.Mapper.cs
src/Controls/src/Core/PlatformConfiguration/WindowsSpecific/FlyoutPage.cs

kubaflo

Could you please add a snapshot?

devanathan-vaithiyanathan · 2026-03-30T07:09:51Z

Could you please add a snapshot?

@kubaflo , CollapseStyle issue was resolved in PR #29927. We also need those changes for collapsedPaneWidth to work properly. I will add an image once PR #29927 is merged.

kubaflo · 2026-03-30T12:40:02Z

@devanathan-vaithiyanathan okay! Merged

kubaflo

Could you please review the AI's comments?

devanathan-vaithiyanathan added 5 commits January 27, 2026 19:28

fix added

6e23ba8

Update FlyoutPage.cs

6feed3f

Update FlyoutPage.Mapper.cs

e7827f6

test added

430baf9

test case renamed

29c355a

dotnet-policy-service bot added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Jan 30, 2026

Update Issue33785.cs

6dd3213

karthikraja-arumugam added the community ✨ Community Contribution label Jan 30, 2026

Tamilarasan-Paranthaman added platform/windows area-controls-flyoutpage FlyoutPage labels Jan 30, 2026

build-analysis bot mentioned this pull request Jan 30, 2026

Project build failure causing test run abort in RunOniOS_BlazorRelease on mac_runios_blazor_release job #33744

Closed

sheiksyedm marked this pull request as ready for review January 30, 2026 13:31

Copilot AI review requested due to automatic review settings January 30, 2026 13:31

Copilot started reviewing on behalf of sheiksyedm January 30, 2026 13:31 View session

Copilot AI reviewed Jan 30, 2026

View reviewed changes

This was referenced Jan 30, 2026

File upload failed even after retry dotnet/dnceng#1916

Open

CI UItests failure System.TimeoutException : Timed out waiting for element... #33220

Open

System.TimeoutException #33245

Open

MauiBot added s/agent-changes-requested AI agent recommends changes - found a better alternative or issues s/agent-fix-win AI found a better alternative fix than the PR s/agent-reviewed PR was reviewed by AI agent workflow (full 4-phase review) labels Mar 18, 2026

devanathan-vaithiyanathan changed the title ~~[Windows] Fix FlyoutPage CollapsedPaneWidth is not working~~ [Windows] FlyoutPage: Fix CollapsedPaneWidth mapping and updates Mar 19, 2026

mauibot concern addressed

43386ab

build-analysis bot mentioned this pull request Mar 19, 2026

"We stopped hearing from agent Azure Pipelines 32. Verify the agent machine is running and has a healthy network connection" dotnet/dnceng#1886

Open

3 tasks

MauiBot added s/agent-gate-failed AI could not verify tests catch the bug s/agent-fix-pr-picked AI could not beat the PR fix - PR is the best among all candidates and removed s/agent-fix-win AI found a better alternative fix than the PR labels Mar 20, 2026

MauiBot added s/agent-review-incomplete AI agent could not complete all phases (blocker, timeout, error) and removed s/agent-changes-requested AI agent recommends changes - found a better alternative or issues labels Mar 28, 2026

kubaflo requested changes Mar 28, 2026

View reviewed changes

MauiBot added s/agent-changes-requested AI agent recommends changes - found a better alternative or issues and removed s/agent-review-incomplete AI agent could not complete all phases (blocker, timeout, error) labels Apr 3, 2026

kubaflo requested changes Apr 3, 2026

View reviewed changes

Conversation

devanathan-vaithiyanathan commented Jan 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue Details

Description of Change

Issues Fixed

Uh oh!

sheiksyedm commented Jan 30, 2026

Uh oh!

azure-pipelines bot commented Jan 30, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

MauiBot commented Mar 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤖 AI Summary

Key Findings

Fix Candidates

Fix Candidates

Cross-Pollination

Uh oh!

MauiBot commented Mar 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤖 AI Summary

Key Findings

Fix Candidates

Fix Candidates

Cross-Pollination

⚠️ Final Recommendation: REQUEST CHANGES

Phase Status

Summary

Root Cause

Fix Quality

Required Changes

Uh oh!

github-actions bot commented Mar 19, 2026

Uh oh!

kubaflo commented Mar 19, 2026

Uh oh!

kubaflo commented Mar 19, 2026

Uh oh!

azure-pipelines bot commented Mar 19, 2026

Uh oh!

MauiBot commented Mar 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🚦 Gate - Test Before and After Fix

Gate Result: ❌ FAILED

Uh oh!

kubaflo left a comment

Choose a reason for hiding this comment

Uh oh!

devanathan-vaithiyanathan commented Mar 30, 2026

Uh oh!

kubaflo commented Mar 30, 2026

Uh oh!

kubaflo left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

devanathan-vaithiyanathan commented Jan 30, 2026 •

edited

Loading

MauiBot commented Mar 18, 2026 •

edited

Loading

MauiBot commented Mar 18, 2026 •

edited

Loading

MauiBot commented Mar 28, 2026 •

edited

Loading