[iOS & Mac] Fix for SearchHandler retains previous page state when switching top tabs

[BagavathiPerumal]

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 navigating between top-level Shell tabs, the SearchHandler does not update to match the currently selected page. Instead, the SearchHandler from the previously selected tab remains visible and active, showing outdated placeholder text, stale search results, and incorrect interaction state.

This issue occurs specifically on iOS and MacCatalyst and is caused by a regression introduced in a recent Shell lifecycle update. The page lifecycle events (Appearing / Disappearing) no longer fire in the correct order, preventing the toolbar from refreshing when a new ShellContent becomes active. As a result, the SearchHandler associated with the new page is never applied.

Root Cause

The issue occurs on iOS and macCatalyst because a recent Shell lifecycle update moved the _tracker.Page assignment to run before the navigation animation completes. As a result, the old page’s Disappearing event was unsubscribed prematurely, preventing it from firing. This left _isVisiblePage stuck in the true state.

When the new page appeared, SetAppeared() exited early due to the stale _isVisiblePage value, causing UpdateShellToMyPage() to never execute. This disrupted the normal Shell page lifecycle and prevented the toolbar from updating. As a result, the SearchHandler, TitleView, and ToolbarItems from the previous tab remained visible instead of being replaced with the new page’s toolbar state.

Regression PR: #33195

Description of Change

The fix restores the correct lifecycle state by invoking SetDisappeared() at the beginning of the if (oldPage != null) block in OnPageSet. This call is additionally guarded with if (newPage != null) to ensure it does not run during Dispose, where Page is set to null and cleanup is handled separately.

When a tab switch assigns _tracker.Page before the animation completes, the call to SetDisappeared() explicitly resets _isVisiblePage to false and unsubscribes Toolbar.PropertyChanged for the outgoing page. After that, the subsequent CheckAppeared() detects that Shell.CurrentPage == newPage and triggers SetAppeared(). With _isVisiblePage now correctly reset, SetAppeared() runs its full logic: updating the SearchHandler, TitleView, and ToolbarItems for the incoming page and re-subscribing to Toolbar.PropertyChanged.

SetDisappeared() is already protected by if (!_isVisiblePage) return;, so it becomes a no-op for normal lifecycle flows where the page already disappeared correctly. No other navigation behavior is affected: push/pop navigation creates a new tracker for each page (meaning oldPage is always null), and app foreground/background cycles do not pass through OnPageSet at all.

Validated the behavior in the following platforms

• Android
• Windows
• iOS
• Mac

Issues Fixed

Fixes #34693

Output ScreenShot

Before	After
34693-BeforeFix.mov	34693-AfterFix.mov

[github-actions]

🚀 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 -- 34735

Or

• Run remotely in PowerShell:

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

[MauiBot]

🚦 Gate - Test Before and After Fix

📊 Expand Full Gate — f4d22ef · fix-34693-Made code changes to fix SearchHandler retaining previous page state when switching Shell top tabs on iOS and macCatalyst.

---

Gate Result: ✅ PASSED

Platform: IOS · Base: main · Merge base: 720a9d4a

Test	Without Fix (expect FAIL)	With Fix (expect PASS)
🖥️ Issue34693 Issue34693	✅ FAIL — 199s	✅ PASS — 84s

🔴 Without fix — 🖥️ Issue34693: FAIL ✅ · 199s

  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 501 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 501 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 4.82 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/Foldable/src/Controls.Foldable.csproj (in 5.87 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.HostApp/Controls.TestCases.HostApp.csproj (in 5.88 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/BlazorWebView/src/Maui/Microsoft.AspNetCore.Components.WebView.Maui.csproj (in 5.88 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 5.88 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Xaml/Controls.Xaml.csproj (in 5.88 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 5.89 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/Maps/src/Controls.Maps.csproj (in 5.9 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/maps/src/Maps.csproj (in 5.91 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.60-ci+azdo.13692691
  Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0-ios26.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0-ios26.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0-ios26.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.60-ci+azdo.13692691
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  Maps -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Maps/Debug/net10.0-ios26.0/Microsoft.Maui.Maps.dll
  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.60-ci+azdo.13692691
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  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
  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
  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:01:39.42
  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/tests/CustomAttributes/Controls.CustomAttributes.csproj (in 765 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.Core/UITest.Core.csproj (in 757 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 757 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 766 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/VisualTestUtils/VisualTestUtils.csproj (in 2 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 814 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 850 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 304 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.NUnit/UITest.NUnit.csproj (in 1.46 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.Appium/UITest.Appium.csproj (in 1.83 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/UITest.Analyzers/UITest.Analyzers.csproj (in 2.33 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/TestUtils/src/VisualTestUtils.MagickNet/VisualTestUtils.MagickNet.csproj (in 2.8 sec).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/tests/TestCases.iOS.Tests/Controls.TestCases.iOS.Tests.csproj (in 2.82 sec).
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  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.60-ci+azdo.13692691
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  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.60-ci+azdo.13692691
  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.13]   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
>>>>> 3/30/2026 3:25:24 PM FixtureSetup for Issue34693(iOS)
>>>>> 3/30/2026 3:25:28 PM ShouldUpdateSearchViewOnPageNavigation Start
>>>>> 3/30/2026 3:25:29 PM ShouldUpdateSearchViewOnPageNavigation Stop
>>>>> 3/30/2026 3:25:29 PM Log types: syslog, crashlog, performance, safariConsole, safariNetwork, server
  Failed ShouldUpdateSearchViewOnPageNavigation [1 s]
  Error Message:
     SearchHandler should show DogsPage placeholder after tab switch
Assert.That(placeholderText, Does.Contain("dog").IgnoreCase)
  Expected: String containing "dog", ignoring case
  But was:  "Enter cat's name"

  Stack Trace:
     at Microsoft.Maui.TestCases.Tests.Issues.Issue34693.ShouldUpdateSearchViewOnPageNavigation() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue34693.cs:line 24

1)    at Microsoft.Maui.TestCases.Tests.Issues.Issue34693.ShouldUpdateSearchViewOnPageNavigation() in /_/src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue34693.cs:line 24


NUnit Adapter 4.5.0.0: Test execution complete

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

🟢 With fix — 🖥️ Issue34693: PASS ✅ · 84s

  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 322 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 329 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 333 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 369 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 383 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.60-ci+azdo.13692691
  Graphics -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Graphics/Debug/net10.0-ios26.0/Microsoft.Maui.Graphics.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0-ios26.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  Core -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Core/Debug/net10.0-ios26.0/Microsoft.Maui.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  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.60-ci+azdo.13692691
  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.60-ci+azdo.13692691
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  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
  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
  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:43.29
  Determining projects to restore...
  Restored /Users/cloudtest/vss/_work/1/s/src/Essentials/src/Essentials.csproj (in 338 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/BindingSourceGen/Controls.BindingSourceGen.csproj (in 323 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Graphics/src/Graphics/Graphics.csproj (in 302 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Core/src/Core.csproj (in 357 ms).
  Restored /Users/cloudtest/vss/_work/1/s/src/Controls/src/Core/Controls.Core.csproj (in 378 ms).
  8 of 13 projects are up-to-date for restore.
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  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.60-ci+azdo.13692691
  Essentials -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/Essentials/Debug/net10.0/Microsoft.Maui.Essentials.dll
  ##vso[build.updatebuildnumber]10.0.60-ci+azdo.13692691
  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.60-ci+azdo.13692691
  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.Appium -> /Users/cloudtest/vss/_work/1/s/artifacts/bin/UITest.Appium/Debug/net10.0/UITest.Appium.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.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.03]   Discovering: Controls.TestCases.iOS.Tests
[xUnit.net 00:00:00.12]   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
>>>>> 3/30/2026 3:26:48 PM FixtureSetup for Issue34693(iOS)
>>>>> 3/30/2026 3:26:53 PM ShouldUpdateSearchViewOnPageNavigation Start
>>>>> 3/30/2026 3:26:54 PM ShouldUpdateSearchViewOnPageNavigation Stop
  Passed ShouldUpdateSearchViewOnPageNavigation [1 s]
NUnit Adapter 4.5.0.0: Test execution complete

Test Run Successful.
Total tests: 1
     Passed: 1
 Total time: 17.0075 Seconds

📁 Fix files reverted (2 files)

• eng/pipelines/ci-copilot.yml
• src/Controls/src/Core/Compatibility/Handlers/Shell/iOS/ShellPageRendererTracker.cs

---

[MauiBot]

🤖 AI Summary

📊 Expand Full Review — f4d22ef · fix-34693-Made code changes to fix SearchHandler retaining previous page state when switching Shell top tabs on iOS and macCatalyst.

---

🔍 Pre-Flight — Context & Validation

---

Issue: #34693 - [Shell][iOS & Mac] SearchHandler retains previous page state when switching top tabs
PR: #34735 - [iOS & Mac] Fix for SearchHandler retains previous page state when switching top tabs
Platforms Affected: iOS, MacCatalyst
Files Changed: 1 implementation, 2 test

Key Findings

• Regression from PR [iOS] Fix Shell NavBarIsVisible updates when switching ShellContent #33195: Moved _tracker.Page assignment before navigation animation completes, causing oldPage.Disappearing to be unsubscribed before it fires. This left _isVisiblePage stuck as true.
• Impact: When new page's tracker calls SetAppeared(), it returns early because _isVisiblePage is already true, so UpdateShellToMyPage() never runs — SearchHandler, TitleView, ToolbarItems are not updated.
• Fix: In OnPageSet(), call SetDisappeared() before unsubscribing old page events, guarded by if (newPage is not null) to skip during Dispose.
• SetDisappeared() is idempotent: Protected by if (!_isVisiblePage) return;, so it's a no-op in normal flows where the page already disappeared correctly.
• Test uses #if IOS || MACCATALYST at file level (not ideal per guidelines, but works for platform-scoped tests).
• No inline review comments from maintainers found.
• Milestone: .NET 10 SR7 (labeled on the issue).
• Gate: ✅ PASSED — test FAILS without fix (shows "Enter cat's name" when on DogsPage), PASSES with fix.

Fix Candidates

#	Source	Approach	Test Result	Files Changed	Notes
PR	PR #34735	Call SetDisappeared() at start of if (oldPage is not null) block in OnPageSet, guarded by if (newPage is not null)	✅ PASSED (Gate)	ShellPageRendererTracker.cs	Minimal, idempotent, targeted fix

---

🔧 Fix — Analysis & Comparison

---

Fix Candidates

#	Source	Approach	Test Result	Files Changed	Notes
1	try-fix (claude-opus-4.6)	Force in when is already	PASS	1 file (+11/-1)	Bypasses guard instead of resetting state; modifies CheckAppeared() behavior
2	try-fix (claude-sonnet-4.6)	Track field; exits early only when	PASS	1 file (+17/-7)	Self-contained state machine; more complex, extra field needed
3	try-fix (gpt-5.3-codex)	Reset in property setter when page changes	PASS	1 file (+1/-0)	Minimal 1-line change; semantically weaker (conflates page assignment with appearance)
4	try-fix (gpt-5.4, gemini unavailable)	Defer old-page Disappearing unsubscription until event fires via HashSet	PASS	1 file (+22/-2)	Preserves real lifecycle but most complex
PR	PR #34735	Call at start of block in , guarded by	PASSED (Gate)	1 file (+9/-0)	Minimal, semantically correct, idempotent

Cross-Pollination

Model	Round	New Ideas?	Details
claude-opus-4.6	2	NO NEW IDEAS	All viable fix surfaces covered across 5 attempts

Exhausted: Yes
Selected Fix: PR # The PR's fix is the cleanest and most semantically correct: calling SetDisappeared() is accurate (the page IS disappearing, the event just didn't fire), it's idempotent (no-op in normal flows), minimal (+9 lines), well-guarded for Dispose, and well-commented. All 4 try-fix alternatives passed but are either more complex (attempts 2, 4), modify a wider surface (attempt 1), or are semantically weaker (attempt 3).34735

---

📋 Report — Final Recommendation

---

✅ Final Recommendation: APPROVE

Phase Status

Phase	Status	Notes
Pre-Flight	✅ COMPLETE	Issue #34693, iOS/MacCatalyst regression from PR #33195
Gate	✅ PASSED	iOS — test FAILs without fix, PASSes with fix
Try-Fix	✅ COMPLETE	4 attempts, 4 passing; Selected Fix: PR
Report	✅ COMPLETE

Summary

PR #34735 fixes a regression on iOS/MacCatalyst where the Shell SearchHandler (and TitleView/ToolbarItems) failed to update when switching between top-level tabs. The fix is a 9-line surgical change to ShellPageRendererTracker.cs. Gate verified the fix works. Four independent try-fix attempts all passed with alternative approaches, confirming the bug is real and fixable — and all alternatives were more complex or semantically weaker than the PR's approach.

Root Cause

A regression in PR #33195 moved _tracker.Page assignment before the navigation animation completes. This caused OnPageSet to unsubscribe oldPage.Disappearing before it fires, leaving _isVisiblePage stuck as true. When the new page's tracker calls CheckAppeared() → SetAppeared(), the early-exit guard (if (_isVisiblePage) return;) prevents UpdateShellToMyPage() from running, so the old tab's SearchHandler/TitleView/ToolbarItems remain visible.

Fix Quality

The PR's fix is well-designed:

• Semantically correct: Calling SetDisappeared() accurately reflects the state — the page IS disappearing (the event just didn't fire due to timing).
• Idempotent: SetDisappeared() has its own if (!_isVisiblePage) return; guard, making it a no-op in normal flows where Disappearing already fired correctly.
• Safely scoped: The if (newPage is not null) guard prevents the call during Dispose (when Page is set to null and cleanup is handled separately).
• Minimal: Only 9 lines added in one targeted location.
• Well-commented: The added comments clearly explain the timing issue and why the fix is needed.

Concerns:

• The test file uses a top-level #if IOS || MACCATALYST preprocessor directive rather than restricting via test categories (per UI test guidelines). This is a minor style issue — the test still works correctly and is appropriately scoped.
• The test _34693CatsPage constructor has inconsistent indentation (mixed tabs/spaces in the ItemTemplate grid setup). Minor cosmetic issue.
• The HostApp page class uses FlyoutBehavior = Flyout and FlyoutHeaderBehavior settings that are not necessary for the tab-switching scenario being tested. These are harmless but add noise.

Try-Fix comparison: All 4 alternative approaches passed tests, confirming multiple valid solutions exist. The PR's approach is preferred because it targets the exact point of failure (the lifecycle gap in OnPageSet) and correctly restores state semantics rather than working around them.

---

[BagavathiPerumal]

🤖 AI Summary

📊 Expand Full Review — f4d22ef · fix-34693-Made code changes to fix SearchHandler retaining previous page state when switching Shell top tabs on iOS and macCatalyst.
🔍 Pre-Flight — Context & Validation
Issue: #34693 - [Shell][iOS & Mac] SearchHandler retains previous page state when switching top tabs PR: #34735 - [iOS & Mac] Fix for SearchHandler retains previous page state when switching top tabs Platforms Affected: iOS, MacCatalyst Files Changed: 1 implementation, 2 test

Key Findings

• Regression from PR [iOS] Fix Shell NavBarIsVisible updates when switching ShellContent #33195: Moved _tracker.Page assignment before navigation animation completes, causing oldPage.Disappearing to be unsubscribed before it fires. This left _isVisiblePage stuck as true.
• Impact: When new page's tracker calls SetAppeared(), it returns early because _isVisiblePage is already true, so UpdateShellToMyPage() never runs — SearchHandler, TitleView, ToolbarItems are not updated.
• Fix: In OnPageSet(), call SetDisappeared() before unsubscribing old page events, guarded by if (newPage is not null) to skip during Dispose.
• SetDisappeared() is idempotent: Protected by if (!_isVisiblePage) return;, so it's a no-op in normal flows where the page already disappeared correctly.
• Test uses #if IOS || MACCATALYST at file level (not ideal per guidelines, but works for platform-scoped tests).
• No inline review comments from maintainers found.
• Milestone: .NET 10 SR7 (labeled on the issue).
• Gate: ✅ PASSED — test FAILS without fix (shows "Enter cat's name" when on DogsPage), PASSES with fix.

Fix Candidates

Source Approach Test Result Files Changed Notes

PR PR #34735 Call SetDisappeared() at start of if (oldPage is not null) block in OnPageSet, guarded by if (newPage is not null) ✅ PASSED (Gate) ShellPageRendererTracker.cs Minimal, idempotent, targeted fix
🔧 Fix — Analysis & Comparison

Fix Candidates

Source Approach Test Result Files Changed Notes

1 try-fix (claude-opus-4.6) Force in when is already PASS 1 file (+11/-1) Bypasses guard instead of resetting state; modifies CheckAppeared() behavior
2 try-fix (claude-sonnet-4.6) Track field; exits early only when PASS 1 file (+17/-7) Self-contained state machine; more complex, extra field needed
3 try-fix (gpt-5.3-codex) Reset in property setter when page changes PASS 1 file (+1/-0) Minimal 1-line change; semantically weaker (conflates page assignment with appearance)
4 try-fix (gpt-5.4, gemini unavailable) Defer old-page Disappearing unsubscription until event fires via HashSet PASS 1 file (+22/-2) Preserves real lifecycle but most complex
PR PR #34735 Call at start of block in , guarded by PASSED (Gate) 1 file (+9/-0) Minimal, semantically correct, idempotent

Cross-Pollination

Model Round New Ideas? Details
claude-opus-4.6 2 NO NEW IDEAS All viable fix surfaces covered across 5 attempts
Exhausted: Yes Selected Fix: PR # The PR's fix is the cleanest and most semantically correct: calling SetDisappeared() is accurate (the page IS disappearing, the event just didn't fire), it's idempotent (no-op in normal flows), minimal (+9 lines), well-guarded for Dispose, and well-commented. All 4 try-fix alternatives passed but are either more complex (attempts 2, 4), modify a wider surface (attempt 1), or are semantically weaker (attempt 3).34735

📋 Report — Final Recommendation

✅ Final Recommendation: APPROVE

Phase Status

Phase Status Notes
Pre-Flight ✅ COMPLETE Issue #34693, iOS/MacCatalyst regression from PR #33195
Gate ✅ PASSED iOS — test FAILs without fix, PASSes with fix
Try-Fix ✅ COMPLETE 4 attempts, 4 passing; Selected Fix: PR
Report ✅ COMPLETE

Summary

PR #34735 fixes a regression on iOS/MacCatalyst where the Shell SearchHandler (and TitleView/ToolbarItems) failed to update when switching between top-level tabs. The fix is a 9-line surgical change to ShellPageRendererTracker.cs. Gate verified the fix works. Four independent try-fix attempts all passed with alternative approaches, confirming the bug is real and fixable — and all alternatives were more complex or semantically weaker than the PR's approach.

Root Cause

A regression in PR #33195 moved _tracker.Page assignment before the navigation animation completes. This caused OnPageSet to unsubscribe oldPage.Disappearing before it fires, leaving _isVisiblePage stuck as true. When the new page's tracker calls CheckAppeared() → SetAppeared(), the early-exit guard (if (_isVisiblePage) return;) prevents UpdateShellToMyPage() from running, so the old tab's SearchHandler/TitleView/ToolbarItems remain visible.

Fix Quality

The PR's fix is well-designed:

• Semantically correct: Calling SetDisappeared() accurately reflects the state — the page IS disappearing (the event just didn't fire due to timing).
• Idempotent: SetDisappeared() has its own if (!_isVisiblePage) return; guard, making it a no-op in normal flows where Disappearing already fired correctly.
• Safely scoped: The if (newPage is not null) guard prevents the call during Dispose (when Page is set to null and cleanup is handled separately).
• Minimal: Only 9 lines added in one targeted location.
• Well-commented: The added comments clearly explain the timing issue and why the fix is needed.

Concerns:

• The test file uses a top-level #if IOS || MACCATALYST preprocessor directive rather than restricting via test categories (per UI test guidelines). This is a minor style issue — the test still works correctly and is appropriately scoped.
• The test _34693CatsPage constructor has inconsistent indentation (mixed tabs/spaces in the ItemTemplate grid setup). Minor cosmetic issue.
• The HostApp page class uses FlyoutBehavior = Flyout and FlyoutHeaderBehavior settings that are not necessary for the tab-switching scenario being tested. These are harmless but add noise.

Try-Fix comparison: All 4 alternative approaches passed tests, confirming multiple valid solutions exist. The PR's approach is preferred because it targets the exact point of failure (the lifecycle gap in OnPageSet) and correctly restores state semantics rather than working around them.

I’ve updated the changes based on the AI summary. Please find the details below:

• The test file uses a top-level #if IOS || MACCATALYST directive instead of test categories. This follows the pattern used in previous testcases and does not affect test execution, so this change can be safely ignored.
• Corrected the inconsistent indentation in the _34693CatsPage constructor’s ItemTemplate section based on the AI suggestion.
• Removed the unnecessary FlyoutBehavior and FlyoutHeaderBehavior properties from the HostApp page to reduce noise and keep the test minimal.

[Copilot]

Pull request overview

Fixes an iOS/MacCatalyst Shell lifecycle regression where switching top tabs can leave the previous page’s toolbar state (notably SearchHandler) active, causing stale placeholder/results and incorrect interaction state.

Changes:

• Updates ShellPageRendererTracker (iOS compatibility handler) to explicitly reset the visible-page lifecycle state during page switches so the incoming page’s toolbar state is applied.
• Adds a HostApp repro Shell with two top tabs (Cats/Dogs) each providing a different SearchHandler placeholder.
• Adds an iOS/MacCatalyst UI test to validate the SearchHandler placeholder updates after tab switching.

Reviewed changes

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

File	Description
src/Controls/src/Core/Compatibility/Handlers/Shell/iOS/ShellPageRendererTracker.cs	Resets tracker lifecycle state on page switches so toolbar/SearchHandler refreshes for the newly selected tab.
src/Controls/tests/TestCases.HostApp/Issues/Issue34693.cs	Adds a minimal Shell top-tab repro with per-tab SearchHandler placeholders to exercise the regression.
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue34693.cs	Adds a targeted iOS/MacCatalyst UITest asserting the SearchField placeholder reflects the newly selected tab.

[Copilot]

After tapping the tab, the test immediately reads the SearchField text. TapTab only waits for the tab element to be present before tapping; it doesn’t guarantee the new page/toolbar state has finished updating. This can make the assertion flaky (especially on iOS/macCatalyst during the transition). Consider waiting for a DogsPage-specific element (e.g., DogPageButton) or waiting until the SearchField text/placeholder contains the expected value before asserting.