-
Notifications
You must be signed in to change notification settings - Fork 1.2k
React 18 support #3108
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 18 support #3108
Conversation
…trum into react-18 # Conflicts: # package.json # packages/@react-aria/combobox/test/useComboBox.test.js # packages/@react-aria/tag/test/useTag.test.js # packages/@react-aria/utils/test/useObjectRef.test.js # packages/@react-spectrum/menu/test/MenuTrigger.test.js # packages/@react-spectrum/table/test/Table.test.js # yarn.lock
Build successful! 🎉 |
Build successful! 🎉 |
Build successful! 🎉 |
Build successful! 🎉 |
Build successful! 🎉 |
# Conflicts: # package.json # packages/@react-aria/dnd/src/useDrag.ts # packages/@react-spectrum/dnd/src/useDragHooks.ts # packages/@react-spectrum/list/src/DragPreview.tsx # packages/@react-spectrum/list/src/ListView.tsx # packages/@react-stately/dnd/src/useDraggableCollectionState.ts # packages/@react-types/shared/src/dnd.d.ts # yarn.lock
Build successful! 🎉 |
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.
Code looks good, approving
Wait until testing session is done for merging
Build successful! 🎉 |
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.
Did a quick sweep of some components/functionality (liveannouncer, drag preview, etc) and things seems to work. Code looks fine to me as well. Looks like there are some conflicts now, but they seem innocuous (ListView one is just the disabledBehavior
stuff from the disabled rows PR)
# Conflicts: # package.json # packages/@react-spectrum/list/src/ListView.tsx # yarn.lock
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.
reapproving, looks fine locally
Build successful! 🎉 |
Closes #2526. Fixes #2800.
This adds basic support for React 18. Strict mode support will be done separately.