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: dragging shadow blocks #7992

Merged
merged 3 commits into from
Apr 4, 2024

Conversation

BeksOmega
Copy link
Collaborator

The basics

The details

Resolves

Fixes N/A

Proposed Changes

#7991 Broke dragging shadow blocks, but this PR fixes it again.

Test Coverage

Tested that dragging shadows, including shadows who are children of shadows, drags the nearest parent real block

Documentation

N/A

Additional Information

N/A

@github-actions github-actions bot added the PR: fix Fixes a bug label Apr 2, 2024
@BeksOmega BeksOmega force-pushed the fix/dragging-shadows branch from c16d490 to f69c396 Compare April 4, 2024 15:56
@BeksOmega BeksOmega marked this pull request as ready for review April 4, 2024 15:58
@BeksOmega BeksOmega requested a review from a team as a code owner April 4, 2024 15:58
@BeksOmega BeksOmega enabled auto-merge (squash) April 4, 2024 18:27
@BeksOmega BeksOmega merged commit c0e6e67 into google:rc/v11.0.0 Apr 4, 2024
6 checks passed
johnnesky pushed a commit to johnnesky/blockly that referenced this pull request Apr 13, 2024
* fix: dragging shadow blocks

* fix: dragging shadows not reverting them to their original position

* fix: shadow parents not being deselected
BeksOmega added a commit to BeksOmega/blockly that referenced this pull request May 16, 2024
BeksOmega added a commit that referenced this pull request May 17, 2024
* Revert "fix: dragging shadow blocks (#7992)"

This reverts commit c0e6e67.

* fix: dragging by shadow not being deletable

* fix: unselecting shadows

* fix: revert changes to select and unselect
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