[Windows, Catalyst] Fix IsPresented=true Not Working on Initial Value in FlyoutPage #31515

devanathan-vaithiyanathan · 2025-09-08T09:58:22Z

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

When setting IsPresented="True" on a FlyoutPage in .NET MAUI , the flyout does not open automatically on app launch as expected.

Description of Change

On macOS, the frame observer connection is delayed using DispatchQueue.MainQueue.DispatchAsync() so XAML properties finish loading before observation starts.

On Windows, PlatformView is loaded before applying the IsPresented value.

Issues Fixed

Fixes #31372

Tested the behavior in the following platforms.

Android
Windows
iOS
Mac

Before	After
Mac	Mac

dotnet-policy-service · 2025-09-08T09:58:41Z

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

Copilot

Pull Request Overview

This PR fixes an issue where setting IsPresented="true" on a FlyoutPage doesn't open the flyout automatically on app launch for Windows and macOS Catalyst platforms. The fix addresses timing issues where platform views are loaded before XAML properties are applied.

Delays frame observer connection on macOS Catalyst using DispatchQueue to ensure XAML properties load first
Adds a Loaded event handler on Windows to apply IsPresented value after platform view is ready
Includes comprehensive UI tests to validate the fix

Reviewed Changes

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

File	Description
`src/Core/src/Handlers/Window/WindowHandler.iOS.cs`	Conditionally delays frame observer connection for macOS Catalyst using DispatchQueue
`src/Core/src/Handlers/FlyoutView/FlyoutViewHandler.Windows.cs`	Adds Loaded event handler to apply IsPresented value after platform view initialization
`src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue31372.cs`	NUnit test to verify flyout is presented on initial load
`src/Controls/tests/TestCases.HostApp/Issues/Issue31372.cs`	UI test page demonstrating FlyoutPage with IsPresented=true

Copilot · 2025-09-08T11:10:05Z

src/Core/src/Handlers/FlyoutView/FlyoutViewHandler.Windows.cs

+		void OnLoaded(object sender, RoutedEventArgs e)
+		{
+			if (VirtualView is not null)
+			{
+				PlatformView.IsPaneOpen = VirtualView.IsPresented;
+			}
 		}


The OnLoaded event handler will fire every time the control is loaded, but the IsPresented synchronization only needs to happen once. Consider unsubscribing from the Loaded event after the first execution to avoid unnecessary repeated executions.

jsuarezruiz · 2025-09-08T16:07:41Z

/azp run MAUI-UITests-public

azure-pipelines · 2025-09-08T16:07:52Z

Azure Pipelines successfully started running 1 pipeline(s).

The base branch was changed.

src/Core/src/Handlers/Window/WindowHandler.iOS.cs

jsuarezruiz · 2025-09-16T06:34:06Z

/azp run

azure-pipelines · 2025-09-16T06:34:30Z

Azure Pipelines successfully started running 3 pipeline(s).

jsuarezruiz · 2025-09-16T06:36:58Z

src/Controls/src/Core/Compatibility/Handlers/FlyoutPage/iOS/PhoneFlyoutPageRenderer.cs

+			if (!_intialLayoutFinished)
+			{
+				_intialLayoutFinished = true;
+				SetInitialPresented();


ViewDidAppear() is much later in the lifecycle. Not a problem, but this could cause a visible delay where the flyout opens after the user sees the page.

@devanathan-vaithiyanathan if you do it via IUIViewLifeCycleEvents does ethat cause less delay?

@PureWeen , PhoneFlyoutPageRenderer is derived from UIViewController, not UIView. Therefore, overriding MovedToWindow isn’t possible, and invoking the MovedToWindow event handler isn’t applicable in this case.

jsuarezruiz · 2025-09-18T11:55:40Z

/azp run

azure-pipelines · 2025-09-18T11:56:02Z

Azure Pipelines successfully started running 3 pipeline(s).

PureWeen · 2025-09-18T21:00:03Z

src/Controls/src/Core/PublicAPI/net-maccatalyst/PublicAPI.Shipped.txt

remove the changes to this file

You can fix this by adding a REMOVE inside the unshipped

or you can use the cake script for this locally

dotnet cake --target=publicapi

@PureWeen , Thanks for the suggestion. I’ve moved the changes from Shipped to Unshipped.

PureWeen · 2025-09-20T19:24:32Z

/azp run

azure-pipelines · 2025-09-20T19:24:58Z

Azure Pipelines successfully started running 3 pipeline(s).

* [Testing] Fix for flaky UITests CookiesCorrectlyLoadWithMultipleWebViews in CI (#31823) This PR addresses the UI test image failures that occurred in the main branch and includes updates to improve rendering and test stability across platforms. - In the CookiesCorrectlyLoadWithMultipleWebViews test case, I changed the WebView source to match the cookies. ### Test cases: - CookiesCorrectlyLoadWithMultipleWebViews * [iOS] Fix added for Navigation.SetTitleView does not work in ios 26. (#31831) * [iOS] Fix added for Navigation.SetTitleView does not work in ios 26. * changes added. * changed the version condition. * [testing] Update androidApiLevels for device tests (#31827) * Update androidApiLevels for device tests * [ci] Update provisioning of api 36 * Add reference (#31669) * [Windows, Catalyst] Fix IsPresented=true Not Working on Initial Value in FlyoutPage (#31515) * mac fix added * Windows fix added * test case added * Update Issue31372.cs * Update Issue31372.cs * Update WindowHandler.iOS.cs * fix updated * Update WindowHandler.iOS.cs * Update WindowHandler.iOS.cs * Update PublicAPI.Shipped.txt * changes added in UnShipped file * Temporarily disable flaky WebView tests using FlakyTest attribute (#31868) * Initial plan * Temporarily disable flaky WebView tests with FlakyTest attribute Co-authored-by: rmarinho <1235097+rmarinho@users.noreply.github.com> * Add tracking issue link to FlakyTest attributes Co-authored-by: rmarinho <1235097+rmarinho@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: rmarinho <1235097+rmarinho@users.noreply.github.com> * Invert NavigationPage UseMauiHandler flag (#31859) * Update dependencies from https://github.com/dotnet/xharness build 20251004.2 (#31872) On relative base path root Microsoft.DotNet.XHarness.CLI , Microsoft.DotNet.XHarness.TestRunners.Common , Microsoft.DotNet.XHarness.TestRunners.Xunit From Version 9.0.0-prerelease.25426.1 -> To Version 9.0.0-prerelease.25504.2 Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com> * [release/9.0.1xx-sr12] Fix ProjectList not displaying due to missing BindingContext update in template sample (#31954) * update ProjectListPage * Update MauiApp.1.csproj --------- Co-authored-by: Vignesh-SF3580 <102575140+Vignesh-SF3580@users.noreply.github.com> * [release/9.0.1xx-sr12] [Android] -Picker dialog causes crash when page is popped while dialo… (#31919) * [Android] -Picker dialog causes crash when page is popped while dialog is open * removed the dispose call. * Revert "removed the dispose call." This reverts commit e1e7a9f. * removed the dispose call. * Modified with Dismiss() method. --------- Co-authored-by: sheiksyedm <sheiksyedm@syncfusion.com> * [release/9.0.1xx-sr12] [iOS] Fix memory leak in CollectionViewHandler2 ItemsLayout PropertyChanged subscription (#31922) * [release/9.0.1xx-sr12] [Android][Regression] Fixed Entry and Editor AppThemeBinding colors for text and placeholder reset to default on theme change (#31921) * added snapshots and fix * removed test cases * added the test cases and snapshots * added braces * added mac and windows snapshots * added mac and windows snapshots * added snapshot --------- Co-authored-by: TamilarasanSF4853 <tamilarasan.velu@syncfusion.com> * 9.0.120 Release Branding (#31979) * [release/9.0.1xx-sr12] [ci] Update arcade main (#31987) * Update arcade * Don t use internal feeds --------- Co-authored-by: Rui Marinho <me@ruimarinho.net> * Update FlyoutViewHandler.Windows.cs (#32007) Co-authored-by: devanathan-vaithiyanathan <114395405+devanathan-vaithiyanathan@users.noreply.github.com> --------- Co-authored-by: TamilarasanSF4853 <tamilarasan.velu@syncfusion.com> Co-authored-by: sheiksyedm <sheiksyedm@syncfusion.com> Co-authored-by: Rui Marinho <me@ruimarinho.net> Co-authored-by: MartyIX <203266+MartyIX@users.noreply.github.com> Co-authored-by: devanathan-vaithiyanathan <114395405+devanathan-vaithiyanathan@users.noreply.github.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: rmarinho <1235097+rmarinho@users.noreply.github.com> Co-authored-by: Tim Miller <drasticactions@users.noreply.github.com> Co-authored-by: dotnet-maestro[bot] <42748379+dotnet-maestro[bot]@users.noreply.github.com> Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Vignesh-SF3580 <102575140+Vignesh-SF3580@users.noreply.github.com>

devanathan-vaithiyanathan added 5 commits September 2, 2025 13:26

mac fix added

cacf32f

Windows fix added

ba743b9

test case added

7f5db87

Update Issue31372.cs

c6c7b88

Update Issue31372.cs

0e7a7bf

dotnet-policy-service bot added the community ✨ Community Contribution label Sep 8, 2025

dotnet-policy-service bot added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Sep 8, 2025

Update WindowHandler.iOS.cs

37fc2b2

devanathan-vaithiyanathan changed the base branch from fix-31372 to main September 8, 2025 11:06

devanathan-vaithiyanathan marked this pull request as ready for review September 8, 2025 11:09

Copilot AI review requested due to automatic review settings September 8, 2025 11:09

devanathan-vaithiyanathan requested a review from a team as a code owner September 8, 2025 11:09

Copilot AI reviewed Sep 8, 2025

View reviewed changes

jsuarezruiz added the area-controls-flyout Flyout label Sep 8, 2025

PureWeen added this to the .NET 9 SR12 milestone Sep 8, 2025

PureWeen added this to MAUI SDK Ongoing Sep 8, 2025

github-project-automation bot moved this to Todo in MAUI SDK Ongoing Sep 8, 2025

PureWeen added the p/0 Work that we can't release without label Sep 8, 2025

jsuarezruiz previously approved these changes Sep 9, 2025

View reviewed changes

github-project-automation bot moved this from Todo to Approved in MAUI SDK Ongoing Sep 9, 2025

PureWeen changed the base branch from main to inflight/current September 11, 2025 15:59

PureWeen changed the base branch from inflight/current to main September 11, 2025 16:01

PureWeen requested changes Sep 11, 2025

View reviewed changes

src/Core/src/Handlers/Window/WindowHandler.iOS.cs Outdated Show resolved Hide resolved

github-project-automation bot moved this from Approved to Changes Requested in MAUI SDK Ongoing Sep 11, 2025

devanathan-vaithiyanathan added 2 commits September 12, 2025 14:16

fix updated

700afe0

Update WindowHandler.iOS.cs

36128c0

Update WindowHandler.iOS.cs

4597846

jsuarezruiz reviewed Sep 16, 2025

View reviewed changes

Update PublicAPI.Shipped.txt

cf9293d

PureWeen requested changes Sep 18, 2025

View reviewed changes

changes added in UnShipped file

dce9403

PureWeen approved these changes Sep 24, 2025

View reviewed changes

github-project-automation bot moved this from Changes Requested to Approved in MAUI SDK Ongoing Sep 24, 2025

jsuarezruiz approved these changes Sep 24, 2025

View reviewed changes

rmarinho merged commit 082d25a into dotnet:main Oct 5, 2025
126 of 130 checks passed

github-project-automation bot moved this from Approved to Done in MAUI SDK Ongoing Oct 5, 2025

devanathan-vaithiyanathan mentioned this pull request Oct 7, 2025

[Windows] Fix FlyoutPage test case failure on main #31896

Merged

[Windows, Catalyst] Fix IsPresented=true Not Working on Initial Value in FlyoutPage #31515

[Windows, Catalyst] Fix IsPresented=true Not Working on Initial Value in FlyoutPage #31515

Uh oh!

Conversation

devanathan-vaithiyanathan commented Sep 8, 2025 • 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!

dotnet-policy-service bot commented Sep 8, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Sep 8, 2025

Choose a reason for hiding this comment

Uh oh!

jsuarezruiz commented Sep 8, 2025

Uh oh!

azure-pipelines bot commented Sep 8, 2025

Uh oh!

Uh oh!

jsuarezruiz commented Sep 16, 2025

Uh oh!

azure-pipelines bot commented Sep 16, 2025

Uh oh!

jsuarezruiz Sep 16, 2025

Choose a reason for hiding this comment

Uh oh!

PureWeen Sep 16, 2025

Choose a reason for hiding this comment

Uh oh!

devanathan-vaithiyanathan Sep 17, 2025

Choose a reason for hiding this comment

Uh oh!

jsuarezruiz commented Sep 18, 2025

Uh oh!

azure-pipelines bot commented Sep 18, 2025

Uh oh!

PureWeen Sep 18, 2025

Choose a reason for hiding this comment

Uh oh!

devanathan-vaithiyanathan Sep 19, 2025

Choose a reason for hiding this comment

Uh oh!

PureWeen commented Sep 20, 2025

Uh oh!

azure-pipelines bot commented Sep 20, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

devanathan-vaithiyanathan commented Sep 8, 2025 •

edited

Loading