Skip to content

fix(abstract): preserve initial dragstart target (closes #2054)#2086

Open
timagixe wants to merge 2 commits into
clauderic:mainfrom
timagixe:issues/2054
Open

fix(abstract): preserve initial dragstart target (closes #2054)#2086
timagixe wants to merge 2 commits into
clauderic:mainfrom
timagixe:issues/2054

Conversation

@timagixe

Copy link
Copy Markdown
Contributor

Summary

Closes #2054.

Fixes a regression in @dnd-kit/abstract where event.operation.target was null in onDragStart.

The fix keeps dragstart before dragover, restores the initial droppable target on the dragstart event, and preserves queued setDropTarget() promise behavior.

Changes

  • Restore operation.target on dragstart.
  • Defer initial dragover until after dragstart.
  • Keep queued setDropTarget() pending until deferred dragover finishes.
  • Add regression tests.

Testing

  • bun run build
  • bun test packages/abstract/tests/drag-event-order.test.ts
  • bun test packages/abstract/tests
  • bun run test

@changeset-bot

changeset-bot Bot commented Jun 14, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e90b297

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 10 packages
Name Type
@dnd-kit/abstract Patch
@dnd-kit/collision Patch
@dnd-kit/dom Patch
@dnd-kit/geometry Patch
@dnd-kit/helpers Patch
@dnd-kit/react Patch
@dnd-kit/state Patch
@dnd-kit/vue Patch
@dnd-kit/solid Patch
@dnd-kit/svelte Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new

pkg-pr-new Bot commented Jun 14, 2026

Copy link
Copy Markdown

Open in StackBlitz

@dnd-kit/abstract

npm i https://pkg.pr.new/@dnd-kit/abstract@2086

@dnd-kit/collision

npm i https://pkg.pr.new/@dnd-kit/collision@2086

@dnd-kit/dom

npm i https://pkg.pr.new/@dnd-kit/dom@2086

@dnd-kit/geometry

npm i https://pkg.pr.new/@dnd-kit/geometry@2086

@dnd-kit/helpers

npm i https://pkg.pr.new/@dnd-kit/helpers@2086

@dnd-kit/react

npm i https://pkg.pr.new/@dnd-kit/react@2086

@dnd-kit/solid

npm i https://pkg.pr.new/@dnd-kit/solid@2086

@dnd-kit/state

npm i https://pkg.pr.new/@dnd-kit/state@2086

@dnd-kit/svelte

npm i https://pkg.pr.new/@dnd-kit/svelte@2086

@dnd-kit/vue

npm i https://pkg.pr.new/@dnd-kit/vue@2086

commit: e90b297

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.

event.operation.target is null on onDragStart Event

1 participant