-
Notifications
You must be signed in to change notification settings - Fork 47.3k
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
Build React DOM Fiber package #7173
Conversation
4375715
to
f28b2c9
Compare
f28b2c9
to
eb78669
Compare
Let's warn on require time for ReactDOMFiber so it's obvious that this is an unsupported API. |
This builds a `react-dom-fiber.js` bundle which exposes ReactDOMFiber. This allows early experiments with the new Fiber reconciler. I also expose it in the npm package through `react-dom/fiber`.
eb78669
to
4e36bf3
Compare
So… do we want to ship this publicly yet? It's definitely unsupported but people might get the wrong idea and think we're going to support this soon. |
If you try to use it, it prints a pretty aggressive warning. I think it's fine to ship CommonJS support but not publish (or be quiet about) UMDs. |
warning( | ||
warned, | ||
'You are using React DOM Fiber which is an experimental renderer. ' + | ||
'It is likely to have bugs, breaking changes and is unsupported.' |
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.
This is the warning that fires if you try to use it.
This builds a `react-dom-fiber.js` bundle which exposes ReactDOMFiber. This allows early experiments with the new Fiber reconciler. I also expose it in the npm package through `react-dom/fiber`. (cherry picked from commit f7e0db9)
Builds on top of #7164
This builds a
react-dom-fiber.js
bundle which exposes ReactDOMFiber. This allows early experiments with the new Fiber reconciler. I now enable it in the Fiber example.I also expose it in the npm package through
react-dom/fiber
.