[timdoege]

Summary

_beginBackPageNavigation marks every descendant outlet as navigating back, but only an outlet that is activated during that back clears its own mark.
The outlets of a page lower in the back stack (for example the per-tab outlets of a tab page) are never activated, so they keep the mark.
A later forward navigation into that page then runs as a back navigation for those outlets, and they render blank.

 _beginBackPageNavigation(frame)
   outlet.setOutletKeyNavigatingBack(outletKey)
-  this.outlets.filter(<is descendant of outlet>).forEach(o => o.isPageNavigationBack = true)
+  getDescendantOutlets(outlet).forEach(o => o.isPageNavigationBack = true)

 _beginPageNavigation(frame, options)
   currentOutlet = getOutletByFrame(frame) || currentOutlet
+  // a forward page navigation means no back is in progress below this outlet
+  getDescendantOutlets(currentOutlet).forEach(o => o._navigatingBackOutlets.clear())
   lastState = currentOutlet.peekState()

The descendant walk is extracted into getDescendantOutlets unchanged, so both call sites share it.

Closes #180

Evidence

Reproduction: https://github.com/timdoege/ns-angular22-tab-outlets-blank-after-back (home -> /tabs -> /detail -> /detail2 -> back -> /tabs again).

Expected (first /tabs)	Before the fix (second /tabs)

• Before: .maestro/repro.yaml fails on 22.0.1 at the last "TAB A CONTENT" check, on iOS and Android.
  After: .maestro/repro.yaml passes on iOS and Android with this change.

.maestro/control.yaml (same steps without the back) passes on 22.0.1, so the back is the trigger.
With the fix, going back from the second tabs page to the first one still shows both tabs.

Merge Danger

Door: two-way

Pure runtime fix in the legacy router, no API or persisted state change; reverting the commit restores the old behavior.

Blast Radius: router

Only affects outlets below the outlet starting a forward page navigation.
Any back mark on those outlets is cleared, so an app that starts a forward page navigation while a nested back is still in flight would see that nested back treated as forward.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: a09b1cc4-0a63-4f71-b20d-b65f80d6d2de

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/NativeScript/angular/@nativescript/angular@181

npm i https://pkg.pr.new/NativeScript/angular/@nativescript/zone-js@181

commit: 72fe1cb