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
Copy Markdown
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
Copy Markdown
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