Skip to content

Commit

Permalink
Fix ReactInstanceManager to initialize the UIManager (facebook#41594)
Browse files Browse the repository at this point in the history
Summary:
Adding `initialize()` to FabricUIManager just as was done by JSIModule

Without this change switching to UIManagerProvider would cause the UI to be Frozen and the events not correctly registered.

Pull Request resolved: facebook#41594

Reviewed By: javache

Differential Revision: D51456979

fbshipit-source-id: 8d97533340a88ec6bb2bf0f257b6acfaa59da471
  • Loading branch information
arushikesarwani94 authored and facebook-github-bot committed Nov 22, 2023
1 parent 4e92016 commit 0f239d3
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -1408,7 +1408,9 @@ private ReactApplicationContext createReactContext(
}
if (ReactFeatureFlags.enableFabricRenderer) {
if (mUIManagerProvider != null) {
catalystInstance.setFabricUIManager(mUIManagerProvider.createUIManager(reactContext));
UIManager uiManager = mUIManagerProvider.createUIManager(reactContext);
uiManager.initialize();
catalystInstance.setFabricUIManager(uiManager);
} else {
catalystInstance.getJSIModule(JSIModuleType.UIManager);
}
Expand Down

0 comments on commit 0f239d3

Please sign in to comment.