Skip to content

fix: position dragging block above/below all blocks#9972

Merged
mikeharv merged 3 commits into
RaspberryPiFoundation:v13from
mikeharv:move-below-all
Jun 10, 2026
Merged

fix: position dragging block above/below all blocks#9972
mikeharv merged 3 commits into
RaspberryPiFoundation:v13from
mikeharv:move-below-all

Conversation

@mikeharv

@mikeharv mikeharv commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

The basics

The details

Proposed Changes

When a block is moved to the workspace in constrained move mode, we will now move it above or below all blocks on the workspace.
image

Reason for Changes

Previously, the fallback placement (above or below the block) was based on the bounding rectangle of a single “root” connected block. This could lead to the dragging block being placed on top of other blocks.

image

@mikeharv mikeharv requested a review from a team as a code owner June 10, 2026 17:27
@mikeharv mikeharv requested a review from maribethb June 10, 2026 17:27
@github-actions github-actions Bot added the PR: fix Fixes a bug label Jun 10, 2026
Comment thread packages/blockly/core/dragging/block_drag_strategy.ts Outdated
@mikeharv mikeharv merged commit dce3315 into RaspberryPiFoundation:v13 Jun 10, 2026
4 checks passed
@mikeharv mikeharv deleted the move-below-all branch June 10, 2026 18:53
@github-actions github-actions Bot added PR: fix Fixes a bug and removed PR: fix Fixes a bug labels Jun 10, 2026
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.

2 participants