Skip to content
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

React Fiber support #484

Closed
maschwenk opened this issue Aug 23, 2017 · 8 comments
Closed

React Fiber support #484

maschwenk opened this issue Aug 23, 2017 · 8 comments

Comments

@maschwenk
Copy link

maschwenk commented Aug 23, 2017

It appears that this library is not compatible with React Fiber. It is using some deprecated/removed React API's. Are there any plans in place to get start this work? Curious how much work is involved.

facebook/react#10294 (comment)

@diasbruno
Copy link
Collaborator

@maschwenk Yeah, that's for sure. Thanks for the link and PRs are always welcome.

@maschwenk
Copy link
Author

maschwenk commented Aug 23, 2017

Initial investigation shows that we need to replace unstable_renderSubtreeIntoContainer with unstable_createPortal. React maintainer details it here: facebook/react#9808 (comment) and provided this fiddle https://jsfiddle.net/r442t0mk/

@sandys
Copy link

sandys commented Aug 28, 2017

we are also getting impacted by this issue.

@diasbruno
Copy link
Collaborator

@maschwenk @sandys I'll merge this issue with @amannn's issue #498 to tackle this. Perhaps, we can open a project and make a roadmap for the next version.

@diasbruno
Copy link
Collaborator

[email protected] is available.

Version 2.3.3 is the stable version and can be installed with react-modal@latest.

To use the compatible version with React 16, you can install by running npm install react-modal@next.

@merlinstardust
Copy link

Why use unstable_createPortal? Why not use ReactDOM.createPortal?

https://egghead.io/lessons/react-render-elements-outside-the-current-react-tree-using-portals-in-react-16

@diasbruno
Copy link
Collaborator

@merlinpatt I think this was before createPortal got in the API.

@diasbruno
Copy link
Collaborator

Released official v3.0.0.

You can follow #498 for more information.

Thank you all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants