Skip to content

[Fix] Wire select option drag handles - #15

Merged
nilzzzzzz merged 1 commit into
mainfrom
codex/wire-select-option-drag-handles
Apr 28, 2026
Merged

[Fix] Wire select option drag handles#15
nilzzzzzz merged 1 commit into
mainfrom
codex/wire-select-option-drag-handles

Conversation

@nilzzzzzz

Copy link
Copy Markdown
Contributor

Reference: n/a

Description

Follow-up to #14: the select option rows already showed drag handles, but the handles were only visual. This wires them to actual sorting.

Solution

  • Wrap select options in dnd-kit sortable context.
  • Attach sortable listeners to the existing drag handle.
  • Keep the visible UI unchanged: handle, input, delete.
  • Add a regression test for drag-end reordering.

Preview (screenshots/screencasts)

n/a

How has this been tested?

  • Unit Tests / Storybook / E2E / Manually
  • npm run test -- FieldConfigDialog
  • npm run lint
  • npm run build

awork Runner config

default

@nilzzzzzz
nilzzzzzz marked this pull request as ready for review April 28, 2026 07:55
@nilzzzzzz
nilzzzzzz merged commit 2fbbfee into main Apr 28, 2026
1 check passed
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.

1 participant