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

chore(form): replace react-sortable-hoc with dndkit #3961

Merged

Conversation

bjoerge
Copy link
Member

@bjoerge bjoerge commented Dec 14, 2022

Description

This replaces react-sortable-hoc with the more modern and capable dndkit for reordering items in arrays in the studio editor.

It's only an internal refactoring and should not affect any customized studios. The UI should for the most part stay the same except for a couple of visual improvements.

What to review

Make sure drag to reorder array items works for:

  • list arrays and grid arrays
  • array of primitive values
  • keyboard navigation & reordering works as it used to

Notes for release

  • n/a

@vercel
Copy link

vercel bot commented Dec 14, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
test-studio ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Dec 16, 2022 at 1:13PM (UTC)
1 Ignored Deployment
Name Status Preview Comments Updated
studio-workshop ⬜️ Ignored (Inspect) Dec 16, 2022 at 1:13PM (UTC)

@bjoerge bjoerge force-pushed the chore/sc-25473/replace-react-sortable-hoc-for-sortable-arrays branch from 3c8fd87 to e602888 Compare December 16, 2022 13:08
@bjoerge bjoerge merged commit 551c4d5 into next Dec 16, 2022
@bjoerge bjoerge deleted the chore/sc-25473/replace-react-sortable-hoc-for-sortable-arrays branch December 16, 2022 13:08
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.

None yet

3 participants