Skip to content

feat(Android, Stack v5): nested stack uses childFragmentManager of parent container#3597

Merged
kkafar merged 2 commits intomainfrom
@kkafar/pop-in-nested-stack
Feb 2, 2026
Merged

feat(Android, Stack v5): nested stack uses childFragmentManager of parent container#3597
kkafar merged 2 commits intomainfrom
@kkafar/pop-in-nested-stack

Conversation

@kkafar
Copy link
Member

@kkafar kkafar commented Jan 30, 2026

Description

Closes https://github.com/software-mansion/react-native-screens-labs/issues/921

StackScreen implements now FragmentProviding interface. This causes
StackContainer of nested stack to use childFragmentManager of parent
container instead of reusing supportFragmentManager.

Changes

N/A

Visual documentation

N/A

Test plan

Not a fix or change in behavior, this is just new behavior.
To verify you need to attach a debugger in StackContainer and verify that it uses
childFragmentManager of parent container.

I've added test example for this use case that I'll likely follow developing in
next PRs.

`StackScreen` implements now `FragmentProviding` interface. This causes
`StackContainer` of nested stack to use `childFragmentManager` of parent
container instead of reusing `supportFragmentManager`.
@kkafar kkafar requested a review from t0maboro January 30, 2026 15:25
Copy link
Contributor

@t0maboro t0maboro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it can be merged, when CI is green

@kkafar
Copy link
Member Author

kkafar commented Feb 2, 2026

It's only linter. I'll land it & fix it with follow up PRs @t0maboro.
I want to avoid all the rebase hustle.

@kkafar kkafar merged commit b6d053c into main Feb 2, 2026
4 of 5 checks passed
@kkafar kkafar deleted the @kkafar/pop-in-nested-stack branch February 2, 2026 13:42
@kkafar kkafar restored the @kkafar/pop-in-nested-stack branch February 2, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants