Skip to content

change(ios): make root VC notify on viewDidAppear/-Disappear#56618

Open
hannojg wants to merge 7 commits intofacebook:mainfrom
hannojg:hannojg/add-viewcontroller-events-to-root-controller
Open

change(ios): make root VC notify on viewDidAppear/-Disappear#56618
hannojg wants to merge 7 commits intofacebook:mainfrom
hannojg:hannojg/add-viewcontroller-events-to-root-controller

Conversation

@hannojg
Copy link
Copy Markdown
Contributor

@hannojg hannojg commented Apr 27, 2026

Summary:

Caution

This is based on the following PR which should be merged first:

this is an example for how to use this pattern. It will enable view components to listen to viewDidAppear: events from their parent UIViewControllers.
For react-native core itself this isn't overly relevant, however, for third party navigation libraries such as react-native-screens it is.

Changelog:

[IOS] [CHANGED] - Introduce RCTViewController which implements notification methods for viewDidAppear/-Disappear

Test Plan:

RN Tester works, compiles and runs.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 27, 2026
@github-actions
Copy link
Copy Markdown

Warning

JavaScript API change detected

This PR commits an update to ReactNativeApi.d.ts, indicating a change to React Native's public JavaScript API.

  • Please include a clear changelog message.
  • This change will be subject to additional review.

This change was flagged as: POTENTIALLY_BREAKING

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant