Skip to content

fix: Fix bug that could cause dragged blocks to connect to and delete others - #10285

Merged
gonfunko merged 2 commits into
mainfrom
along-for-the-ride
Aug 18, 2026
Merged

fix: Fix bug that could cause dragged blocks to connect to and delete others#10285
gonfunko merged 2 commits into
mainfrom
along-for-the-ride

Conversation

@gonfunko

@gonfunko gonfunko commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

The basics

The details

Resolves

Fixes #10281

Proposed Changes

This PR fixes a bug where dragged blocks that were dropped onto a delete area could connect to other blocks on the workspace just before they were deleted, causing the non-dragged blocks to be deleted as well.

Test Coverage

I added a browser test to exercise this scenario, and verified that it failed without the change in place.

@gonfunko
gonfunko requested a review from a team as a code owner August 18, 2026 20:23
@gonfunko
gonfunko requested a review from mikeharv August 18, 2026 20:23
@github-actions github-actions Bot added the PR: fix Fixes a bug label Aug 18, 2026
@github-actions github-actions Bot added PR: fix Fixes a bug and removed PR: fix Fixes a bug labels Aug 18, 2026
Comment thread packages/blockly/tests/browser/test/dragger_test.mjs Outdated
@gonfunko
gonfunko merged commit 7aba030 into main Aug 18, 2026
6 checks passed
@gonfunko
gonfunko deleted the along-for-the-ride branch August 18, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: fix Fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dropping blocks into the flyout can delete the blocks behind it

2 participants