Skip to content

Conversation

shwanton
Copy link

@shwanton shwanton commented Jan 24, 2023

Please select one of the following

  • I am removing an existing difference between facebook/react-native and microsoft/react-native-macos 👍
  • I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos 👍
  • I am making a fix / change for the macOS implementation of react-native
  • I am making a change required for Microsoft usage of react-native

Summary

Fabric on macOS implementation:
Fix misc rendering & build issues

closes #1573
closes #1570
closes #1571
closes #1566
closes #1565

Changelog

[macOS][Added] - Fix misc rendering & build issues

Test Plan

[x] Build RNTester-macOS w/ Fabric - doesn’t run yet, but no RCTLegacyViewManagerInteropComponentView errors
CleanShot 2023-01-23 at 21 54 01

[x] Build RNTester - iOS w/ Fabric
CleanShot 2023-01-23 at 21 50 43

[x] Build RNTester-macOS w/ Paper - should work
CleanShot 2023-01-23 at 22 29 03

[x] Build RNTester - iOS w/ Paper -
CleanShot 2023-01-23 at 22 31 40

@shwanton shwanton changed the title Fix RCTLegacyViewManagerInteropComponentView [Fabric] Fix build issues Jan 24, 2023
@shwanton shwanton changed the title [Fabric] Fix build issues [Fabric] Fix misc rendering & build issues Jan 24, 2023
@shwanton shwanton marked this pull request as ready for review January 24, 2023 06:50
@shwanton shwanton requested a review from a team as a code owner January 24, 2023 06:50
…componentview

# Conflicts:
#	React/Fabric/Mounting/ComponentViews/InputAccessory/RCTInputAccessoryComponentView.mm
#	React/Fabric/Mounting/UIView+ComponentViewProtocol.mm
@Saadnajmi
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@Saadnajmi Saadnajmi merged commit 1b4706f into microsoft:main Jan 24, 2023
@shwanton shwanton deleted the fabric/fix-interopcomponentview branch January 24, 2023 23:46
shwanton added a commit to shwanton/react-native-macos that referenced this pull request Feb 13, 2023
* Fix RCTLegacyViewManagerInteropComponentView

* Fix Fabric crash setting frame

* Fix crash w/ first responder

* Fix RCTFindTextInputWithNativeId

Co-authored-by: Shawn Dempsey <shawndempsey@fb.com>
shwanton added a commit to shwanton/react-native-macos that referenced this pull request Mar 10, 2023
* Fix RCTLegacyViewManagerInteropComponentView

* Fix Fabric crash setting frame

* Fix crash w/ first responder

* Fix RCTFindTextInputWithNativeId

Co-authored-by: Shawn Dempsey <shawndempsey@fb.com>
# Conflicts:
#	React/Fabric/Mounting/ComponentViews/LegacyViewManagerInterop/RCTLegacyViewManagerInteropComponentView.mm
shwanton added a commit to shwanton/react-native-macos that referenced this pull request Mar 10, 2023
* Fix RCTLegacyViewManagerInteropComponentView

* Fix Fabric crash setting frame

* Fix crash w/ first responder

* Fix RCTFindTextInputWithNativeId

Co-authored-by: Shawn Dempsey <shawndempsey@fb.com>
# Conflicts:
#	React/Fabric/Mounting/ComponentViews/LegacyViewManagerInterop/RCTLegacyViewManagerInteropComponentView.mm
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants