-
Notifications
You must be signed in to change notification settings - Fork 560
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
Compatibility with React 18? #916
Comments
I'm finding that Listbox doesn't work (no response after clicking on the trigger) in my new project using React 18, but the same setup works in React 17. |
I don't support React 18 just yet. There are a number of things that won't play well with concurrent rendering or the new StrictMode behavior. I'm going to start going through all of that this weekend and take stock of what all needs to be done to get there. |
It would be nice to just relax the |
|
I agree. Basically when you don't include some version in peer dependencies you say "don't even try using it with that version".
This is also the default behavior in NPM 8, which comes with the current Node LTS |
@chaance, would you mind to share what all needs to be done to support React 18 here? |
@jeanpan Main thing is checking everything out in There may be some other things, but running the test suite w/ React 18 and checking the storybook examples would be a good way to find out for sure. |
This comment was marked as off-topic.
This comment was marked as off-topic.
I tried running the new |
|
4 months have passed since the issue was opened and, among the 30 dependencies in my project, @reach/dialog is the only one that is not compatible with react 18.x.x. When will you release the new version approximately? I would love to help, but I definitely don't have the expertise for that. Thank you for all the amazing work! |
Please keep in mind that this is a free and open source project, free for you to use, fork, or abandon if it no longer works for you. It is maintained very sparingly at the moment, whenever I have the time. There are no updates at the moment, so do with that what you will. The best way to move the needle on this issue is by helping me out with tasks noted above. If you don't have the expertise to jump into the actual code, organizational work (breaking down tasks, managing progress, etc.) is also very helpful. This is a great opportunity to invest in a project you rely on, which is the only way [unfunded] open source work can be sustainable. (Alternatively, I am available for limited contract work if you want to pay me 😄) Here is a table of all packages I can use to keep track of what needs to be done. Because we don't have as much test coverage as I'd like, new tests would be appreciated. We'll still need manual testing (particularly around keyboard navigation in packages that rely on
For those who want to help with debugging and developmentI suggest starting with the lower-level packages that all other packages depend on. Identifying and fixing issues in packages like |
I'm start check |
Hey sorry I'm curious, to do all these tests and say that they pass in React 18 what are the changes required? Having React 18 installed as a dependency, change |
tested portal in my app React 18 .
Uncaught Error: Hydration failed because the initial UI does not match what was rendered on the server. |
We are looking at moving to React 18 - Does reach-ui support react 18 now ? Appears to be no doco saying so. |
We are using it without any problem in a nextjs app. |
Works when
|
I plan to migrate to React v18. However, the |
It would be really nice to have React v18 setup for this repo. Projects dependent on this library are blocked on upgrading, or having to fork and update. |
Jumping in here to mention that when trying with React 19 there are runtime warnings now as well 😭
|
Components are not compatible with React 18 as it can be seen in package-lock.json:
Eslint throws warnings, but shows 0 vulnerabilities and I haven't had any issue yet.
Is it safe to use Reach UI with React 18? When will you update the libraries? Thank you in advance!
The text was updated successfully, but these errors were encountered: