Skip to content

[codex] Reset drag position after cancelled stop - #1008

Open
nexiumbiz-debug wants to merge 1 commit into
tiberiuzuld:masterfrom
nexiumbiz-debug:codex/reset-drag-position-on-cancel-694
Open

nexiumbiz-debug wants to merge 1 commit into
tiberiuzuld:masterfrom
nexiumbiz-debug:codex/reset-drag-position-on-cancel-694

Conversation

@nexiumbiz-debug

Copy link
Copy Markdown
Contributor

Summary

  • reset draggable position bookkeeping when a rejected stop callback cancels a drag
  • keep restored $item coordinates and exposed drag.positionX/Y in sync
  • add regression coverage for cancelled drags

Fixes #694

Validation

  • npx prettier --check projects/angular-gridster2/src/lib/gridsterDraggable.ts projects/angular-gridster2/src/lib/tests/gridsterDraggable.spec.ts
  • npx eslint projects/angular-gridster2/src/lib/tests/gridsterDraggable.spec.ts
  • git diff --check
  • npm run test-lib -- --watch=false
  • npm run build-lib

legrottagliegionata pushed a commit to legrottagliegionata/angular-gridster2-plus that referenced this pull request Sep 13, 2026
tiberiuzuld#1008)

Keep positionX/positionY and their backups in sync with the restored item when
draggable.stop rejects, so the next stop callback does not see a stale target.

Fixes upstream tiberiuzuld#694.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.

draggable.stop callback rejection - still contains old drag position

1 participant