Skip to content

Conversation

shwanton
Copy link

@shwanton shwanton commented Dec 9, 2022

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:
Disable UIAccessibilityElement API's till we update them for Fabric

Changelog

[macOS][Added] - Disable UIAccessibilityElement API's till we update them for Fabric

Test Plan

[x] Build RNTester-macOS w/ Fabric - doesn’t run yet, but no RCTAccessibilityElement errors
CleanShot 2022-12-09 at 13 16 49

Build errors:
Build RNTester-macOS_2022-12-09T13-15-18.txt

[x] Build RNTester - iOS w/ Fabric
CleanShot 2022-12-09 at 14 15 55

[x] Build RNTester-macOS w/ Paper - should work
CleanShot 2022-12-09 at 13 42 53

[x] Build RNTester - iOS w/ Paper - should work
CleanShot 2022-12-09 at 13 41 44

@shwanton shwanton marked this pull request as ready for review December 9, 2022 22:17
@shwanton shwanton requested a review from a team as a code owner December 9, 2022 22:17
@analysis-bot
Copy link

analysis-bot commented Dec 15, 2022

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: 9669c10
Branch: main

Copy link
Collaborator

@Saadnajmi Saadnajmi left a comment

Choose a reason for hiding this comment

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

feedback about tag format

@Saadnajmi Saadnajmi merged commit b2c5fa9 into microsoft:main Dec 22, 2022
@shwanton shwanton deleted the fabric/shim-accessibility branch December 28, 2022 19:54
shwanton added a commit to shwanton/react-native-macos that referenced this pull request Feb 13, 2023
… Fabric (microsoft#1581)

* Disable UIAccessibilityElement API's till we can update them for Fabric

* Fix macOS tags

* Fix macOS tags

* Add missing macOS tag

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
… Fabric (microsoft#1581)

* Disable UIAccessibilityElement API's till we can update them for Fabric

* Fix macOS tags

* Fix macOS tags

* Add missing macOS tag

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
… Fabric (microsoft#1581)

* Disable UIAccessibilityElement API's till we can update them for Fabric

* Fix macOS tags

* Fix macOS tags

* Add missing macOS tag

Co-authored-by: Shawn Dempsey <shawndempsey@fb.com>
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.

4 participants