Skip to content

fix: keep drag transform snapshots current#2087

Open
pupuking723 wants to merge 1 commit into
clauderic:mainfrom
pupuking723:fix/feedback-none-transform-snapshot
Open

fix: keep drag transform snapshots current#2087
pupuking723 wants to merge 1 commit into
clauderic:mainfrom
pupuking723:fix/feedback-none-transform-snapshot

Conversation

@pupuking723

@pupuking723 pupuking723 commented Jun 15, 2026

Copy link
Copy Markdown

Fixes #2055

Summary

  • Compute the next drag transform for dragmove event snapshots from the pending move coordinates.
  • Refresh the cached operation transform after the move is committed, so later snapshots such as dragend do not depend on the Feedback plugin reading dragOperation.transform.
  • Add a manager-level regression test for modified transforms when no feedback plugin is involved.
  • Add a patch changeset for @dnd-kit/abstract.

Testing

  • npx bun@1.3.11 test packages/abstract/tests/manager-modifiers.test.ts
  • npx bun@1.3.11 run --cwd packages/abstract build
  • git diff --check

Note: I also tried running ESLint directly on the changed files, but it failed before linting with an ESLint/Ajv initialization error (Cannot set properties of undefined (setting 'defaultMeta')).

@changeset-bot

changeset-bot Bot commented Jun 15, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9852d8d

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

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.transform never gets updated when feedback is "none"

1 participant