-
Notifications
You must be signed in to change notification settings - Fork 31
[Preview] feat: support new architecture with rn 0.79 #48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Just wanted to chip in here to say that I tested this and it works 100% for me! |
This PR has been automatically labelled "stale" because it hasn't had recent activity. |
Hey, thanks so much for the awesome work on this! I’ve been working on upgrading to React Native 0.79, and the PasteInput package was a blocker — this PR made the upgrade possible, so I really appreciate it. Just a heads-up: if your project uses static frameworks (which some libraries like Firebase require), the app won’t compile due to some missing header files. How to Reproduce (with Expo):
Possible Fix: |
@aminerol Ah, some people had an issue with this in another library that I maintain, and I believe this diff is what fixed it. I suppose a similar change to the podspec needs to be made here? |
Hey @haileyok, thanks for prompt response, indeed that diff fixed one missing header file which is related to Although i needed to add one more path to that diff related to This is the final patch that made the project compiles again
|
Summary
This is more of a PR for visibility than it is a PR for actually getting merged. It's fairly messy and touches a lot. It also will be, admittedly, annoying to maintain as e.g. new React Native versions are released. However, it is the only way that I have been able to get the full feature set that worked with Paper to work with Fabric. Someone who is more in tune with e.g. shadow nodes as well as the main React Native
TextInput
implementation might have a cleaner way of getting this to work.We do a few things here:
react-native.config.js
to set theCMakeLists.txt
pathWithout the last step, at least two basic functionalities of a base
<TextInput>
are broken with<PasteTextInput>
:<Text>
components as children of<PasteTextInput>
does not work, for similar reasons (no logic implemented inside of the shadow node for state updates)Again, I don't expect this to get merged as-is, however I hope that it can serve as a guide for someone who has the time and willingness to do it "correctly". Likely:
ShadowNode
logic, but almost certainly can prune down or simply subclass theProps
andCompoenentDescriptor
?Props
could be shared.I am personally willing - if I find more time... - to implement any suggestions others have along these lines if feedback is left.
I've tagged https://github.com/bluesky-social/react-native-paste-input with the tag
rn-079
for those who wish to test this themselves.Ticket Link
#47