-
Notifications
You must be signed in to change notification settings - Fork 171
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
plans re: React v16 #163
Comments
So my plan is to significantly simplify this library (there is already some code in master). Having only basic minimalistic wrapper for I am still not sure if it's valuable to keep/develop stateful version of portal. It's nice for some basic use-cases but people tend to overuse it, asking for various life-cycle hooks while they should be rather using stateless version and implement whatever they need around it. Opinions welcome! |
Obviously this library is pretty popular (over 300k downloads/month) so I don't want to cause a crisis but some major changes are needed. |
Check: #157 |
This is a great project that has shaped and informed the development of the ecosystem, which is evidenced by convergent evolution for other frameworks, e.g. https://github.com/LinusBorg/portal-vue and https://github.com/yapplabs/ember-wormhole, and now with the introduction of portals as a core feature in React v16.
Just curious, what is the future of the
react-portal
package?The text was updated successfully, but these errors were encountered: