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.


Reviewed By: javache

Differential Revision: D51456979
  • Loading branch information
arushikesarwani94 authored and facebook-github-bot committed Nov 22, 2023
1 parent d4399c5 commit 47aca51
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 47aca51

Please sign in to comment.