Skip to content

Conversation

@BeksOmega
Copy link
Contributor

The basics

The details

Resolves

Fixes N/A

#4886 never actually worked.

Proposed Changes

Makes it so that shouldPreventMove is actually respected.

Test Coverage

Manually tested by linking to the backpack plugin.

  • Dragging an orphaned block moves it back to the old location on the workspace.
  • Dragging a child block reattaches it to parent.
  • Dragging a parent block (holding ctrl to heal stack) reattaches to child.
  • Dragging a middle block (holding ctrl to heal stack) reattaches to parent and child.
  • Dragging from the simple flyout toolbox bumps blocks into bounds.

Documentation

N/A

Additional Information

I'm going to release a beta of 10.5 for this for app inventor so they can test with it.

@BeksOmega BeksOmega requested a review from a team as a code owner February 28, 2024 19:38
@github-actions github-actions bot added the PR: fix Fixes a bug label Feb 28, 2024
@ewpatton
Copy link
Contributor

I've tested with the beta version @BeksOmega sent and we have deployed in our Blockly update test server.

@BeksOmega BeksOmega merged commit 7c7dfd0 into RaspberryPiFoundation:develop Mar 7, 2024
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.

3 participants