Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix drag and drop target position calculation #201872

Merged
merged 1 commit into from
Jan 5, 2024
Merged

Conversation

benibenj
Copy link
Contributor

@benibenj benibenj commented Jan 5, 2024

This PR addresses corner cases in the drag and drop functionality. The target position calculation has been updated to correctly handle instances where the source position is before or after the target. This ensures that elements are moved to the correct position in the list.

@benibenj benibenj self-assigned this Jan 5, 2024
@benibenj benibenj enabled auto-merge (squash) January 5, 2024 09:44
@VSCodeTriageBot VSCodeTriageBot added this to the December / January 2024 milestone Jan 5, 2024
@benibenj benibenj merged commit d57d309 into main Jan 5, 2024
7 checks passed
@benibenj benibenj deleted the benibenj/teenage-piranha branch January 5, 2024 10:03
@microsoft microsoft locked and limited conversation to collaborators Jun 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants