Skip to content

fix(cdk/drag-drop): item returned to wrong index in initial container#32944

Merged
crisbeto merged 1 commit intoangular:mainfrom
crisbeto:32940/drag-drop-return-logic
Mar 18, 2026
Merged

fix(cdk/drag-drop): item returned to wrong index in initial container#32944
crisbeto merged 1 commit intoangular:mainfrom
crisbeto:32940/drag-drop-return-logic

Conversation

@crisbeto
Copy link
Member

Fixes that when we return the item to its original container, we may end up putting it at the wrong index, because the logic that de-duplicates the item runs too late.

Fixes #32940.

Fixes that when we return the item to its original container, we may end up putting it at the wrong index, because the logic that de-duplicates the item runs too late.

Fixes angular#32940.
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Mar 17, 2026
@pullapprove pullapprove bot requested review from adolgachev and ok7sai March 17, 2026 09:47
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Mar 18, 2026
@crisbeto crisbeto merged commit 0767a3c into angular:main Mar 18, 2026
29 of 32 checks passed
@crisbeto
Copy link
Member Author

This PR was merged into the repository. The changes were merged into the following branches:

crisbeto added a commit that referenced this pull request Mar 18, 2026
…#32944)

Fixes that when we return the item to its original container, we may end up putting it at the wrong index, because the logic that de-duplicates the item runs too late.

Fixes #32940.

(cherry picked from commit 0767a3c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: cdk/drag-drop target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(drag'n'drop): List order changes despite cdkDropListSortingDisabled

2 participants