Fix missing iOS navigation events and removes duplicate page (dis)appearing events

Author: albyrock87

CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Fixes
 
 - Unknown stack frames in profiles on .NET 8+ ([#3967](https://github.com/getsentry/sentry-dotnet/pull/3967))
+- Missing MAUI `Shell` navigation breadcrumbs on iOS ([#4006](https://github.com/getsentry/sentry-dotnet/pull/4006))
 
 ## 5.3.0
 

src/Sentry.Maui/Internal/MauiEventsBinder.cs

@@ -34,7 +34,16 @@ public void HandleApplicationEvents(Application application, bool bind = true)
     {
         if (bind)
         {
-            // Attach element events to all descendents as they are added to the application.
+            // Attach element events to all existing descendants (skip the application itself)
+            foreach (var descendant in application.GetVisualTreeDescendants().Skip(1))
+            {
+                if (descendant is VisualElement element)
+                {
+                    OnApplicationOnDescendantAdded(application, new ElementEventArgs(element));
+                }
+            }
+
+            // Attach element events to all descendants as they are added to the application.
             application.DescendantAdded += OnApplicationOnDescendantAdded;
             application.DescendantRemoved += OnApplicationOnDescendantRemoved;
 

test/Sentry.Maui.Tests/MauiEventsBinderTests.Application.cs

@@ -145,6 +145,36 @@ public void Application_UnbindModalEvents_DoesNotAddBreadcrumb(string eventName,
         Assert.Single(_fixture.Scope.Breadcrumbs);
     }
 
+    [Fact]
+    public void Application_HandleApplicationEvents_TracksExistingDescendants()
+    {
+        // Arrange
+        var application = MockApplication.Create();
+        var element = new MockVisualElement("element");
+        var mainPage = new ContentPage
+        {
+            Content = new VerticalStackLayout
+            {
+                element
+            }
+        };
+
+        application.AddWindow(mainPage);
+
+        _fixture.Binder.HandleApplicationEvents(application);
+
+        // Act
+        element.RaiseEvent(nameof(VisualElement.Focused), new FocusEventArgs(element, true));
+
+        // Assert
+        var crumb = Assert.Single(_fixture.Scope.Breadcrumbs);
+        Assert.Equal($"{nameof(MockVisualElement)}.{nameof(VisualElement.Focused)}", crumb.Message);
+        Assert.Equal(BreadcrumbLevel.Info, crumb.Level);
+        Assert.Equal(MauiEventsBinder.SystemType, crumb.Type);
+        Assert.Equal(MauiEventsBinder.RenderingCategory, crumb.Category);
+        crumb.Data.Should().Contain($"{nameof(MockVisualElement)}.Name", "element");
+    }
+
     public static IEnumerable<object[]> ApplicationModalEventsData
     {
         get

test/Sentry.Maui.Tests/Mocks/MockApplication.cs

@@ -5,6 +5,7 @@ namespace Sentry.Maui.Tests.Mocks;
 public class MockApplication : Application
 {
     private static readonly object LockObj = new();
+    private Page _mainPage;
 
     static MockApplication()
     {
@@ -18,6 +19,23 @@ private MockApplication()
     {
     }
 
+    public void AddWindow(Page mainPage)
+    {
+        _mainPage = mainPage;
+        ((IApplication)this).CreateWindow(null);
+        _mainPage = null;
+    }
+
+    protected override Window CreateWindow(IActivationState activationState)
+    {
+        if (_mainPage != null)
+        {
+            return new Window(_mainPage);
+        }
+
+        return base.CreateWindow(activationState);
+    }
+
     public static MockApplication Create()
     {
         // The base constructor will try to set the mock as the current application, which we don't want in tests.