-
Notifications
You must be signed in to change notification settings - Fork 46.9k
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
expose isHydrating #16909
expose isHydrating #16909
Conversation
ReactDOM: size: 0.0%, gzip: 0.0% Details of bundled changes.Comparing: 911104a...95d4234 react-dom
|
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.
Sure. Any reason we don't want to expose it on the reconciler interface, similar to findHostInstance
?
This change is most likely just going to be temporary and only used in one place, so it seemed easier to put it on the SECRET_INTERNALS object for now so it can also be easily removed. Also, it makes it (hopefully) less likely people will use it. Longer term, I think we (@sebmck) want to expose a hook rather than this field for users. Also, if we put this field only on ReactDomFB, only FB will be able to access it, whereas if we put it anywhere else we potentially are exposing it in other places |
Added a function on
ReactFiberHydrationContext
to exposeisHydrating
inReactDOMFB