Skip to content

Conversation

shwanton
Copy link

@shwanton shwanton commented Nov 19, 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

The PropTypes package is deprecated on RN Core.
To help folks get off of the package, this deprecation path was left in place.

Removing the reference fixes #1433

NOTE: This was recently un-deprecated by the core team to allow more time for folks to migrate: facebook@b966d29
I'm assuming desktop doesn't have the same legacy issues so I think it's fine to remove.

Changelog

[macOS] [Fix] - Remove reference to deprecated package in TextView

Test Plan

Before

CleanShot.2022-11-18.at.17.31.46.mp4

After

CleanShot.2022-11-18.at.17.45.26.mp4

@shwanton shwanton marked this pull request as ready for review November 19, 2022 01:47
@shwanton shwanton requested a review from a team as a code owner November 19, 2022 01:47
@christophpurrer
Copy link

Nice!

@Saadnajmi
Copy link
Collaborator

@amgleitman did you have a reason to keep these last merge? I remember they came up

@amgleitman
Copy link
Member

amgleitman commented Dec 6, 2022

@amgleitman did you have a reason to keep these last merge? I remember they came up

I added the lines that were being deleted as part of fc17709, which corresponds to 3f62904 in RNCore.

The removal of these lines in RNCore happened in 3e229f2, which didn't make it into a stable release until 0.69. That being said, if we want to remove it now and everything still works, then I say go for it. We'll have to get rid of it at some point anyway.

@Saadnajmi Saadnajmi merged commit 7fbfa75 into microsoft:main Dec 9, 2022
@shwanton shwanton deleted the remove-deprecated-prop-types-text branch December 16, 2022 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RN-Tester- Text page shows invalid prop type "apple_fontSmoothing"

4 participants