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 18 support #3108

Merged
merged 20 commits into from May 20, 2022
Merged

React 18 support #3108

merged 20 commits into from May 20, 2022

Conversation

devongovett
Copy link
Member

@devongovett devongovett commented May 6, 2022

Closes #2526. Fixes #2800.

This adds basic support for React 18. Strict mode support will be done separately.

eps1lon and others added 12 commits November 7, 2021 19:24
…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
@adobe-bot
Copy link

@adobe-bot
Copy link

@adobe-bot
Copy link

@adobe-bot
Copy link

@adobe-bot
Copy link

# 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
@snowystinger snowystinger marked this pull request as ready for review May 19, 2022 20:53
@adobe-bot
Copy link

snowystinger
snowystinger previously approved these changes May 19, 2022
Copy link
Member

@snowystinger snowystinger left a 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

@adobe-bot
Copy link

LFDanLu
LFDanLu previously approved these changes May 20, 2022
Copy link
Member

@LFDanLu LFDanLu left a 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
@devongovett devongovett dismissed stale reviews from LFDanLu and snowystinger via e8372e6 May 20, 2022 23:42
dannify
dannify previously approved these changes May 20, 2022
Copy link
Member

@LFDanLu LFDanLu left a 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

@adobe-bot
Copy link

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

Successfully merging this pull request may close these issues.

Support React 18
6 participants