-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
[RFR] Remove FC
usage from Field
components
#6538
Conversation
FC
usage from Field
componentsFC
usage from Field
components
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
I'm afraid we'll have to revert this one. To see why, try to run the demo with
The culprit is DefinitelyTyped/React, which types This bug has been closed without fix: DefinitelyTyped/DefinitelyTyped#36320 So although FC has drawbacks, not using FC for memoized components is worse. |
I'll make a PR to fix it |
Thank you! |
… propTypes Partially reverts #6538
… propTypes Partially reverts marmelab#6538
No description provided.