Update to Blockly 11#3588
Conversation
|
I just found a bug that I'll work on next. While delete count is correct (includes children too) when selecting multiple blocks only the block (and its children, if any) you right click on is deleted. |
|
Is this true in master as well? In either case the issue will be somewhere in the multi select plugin.Sent from my iPhoneOn Sep 5, 2025, at 18:33, Michael Gallagher ***@***.***> wrote:mjgallag left a comment (mit-cml/appinventor-sources#3588)
I just found a bug that I'll work on next. While delete count is correct (includes children too) when selecting multiple blocks only the block (and its children, if any) you right click on is deleted.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
|
@ewpatton it works in master so I either messed up the merge mit-cml/workspace-multiselect@main...mjgallag:workspace-multiselect:update-to-blockly-11 or its a Blockly 11 issue with the plugin. |
|
@ewpatton its reproducible on main branch of multiselect plugin but works if I try commit before it was upgraded to Blockly 11. |
|
@ewpatton ha ... now i'm not so sure about any of my statements regarding this bug ... tbd |
|
Ah ha, it's a subtle bug that was introduced when the multiselect plugin was upgraded to Blockly 11. If you keep the shift key pressed when you right click the yellow borders of multiple select are lost and even though context menu has correct multiple counts it only acts on block you right clicked. If you let go of shift key after making your selections and before right clicking it works as expected. Will look into this but will get test instance up first so people can play with it in parallel with my bug fixing. |
|
https://arched-branch-471517-f9.uc.r.appspot.com/ here is an appspot instance for playing around with this |
8eb92fc to
785162b
Compare
aba67f3 to
ca30347
Compare
|
OK that bug is fixed and https://arched-branch-471517-f9.uc.r.appspot.com/ has been updated. Also there is no more fork for multiselect. I have marked this ready for review. I expect more bugs will probably be found but I think it's ready for more eyes. |
|
I just noticed with https://appinventor.mit.edu/explore/tips/typeblocking the blocks are landing on top of each other in this branch instead of staggered like on master. @josmas I think I am going to try to integrate your plugin https://github.com/josmas/blockly-plugins/tree/feat/typeblocking as part of this Blockly 11 branch. |
9e5b36b to
4903e4d
Compare
|
Nevermind I just fixed the bugs as moving to @josmas typeblocking plugin should be a separate pull. This bug lead me to fix the same bug in a few different places and also lead me to notice paste from backapck was broken, that fix was pretty interesting. In any case https://arched-branch-471517-f9.uc.r.appspot.com/ has been updated. I am off next week and a half. If anyone gets around to testing this please leave bugs in the comments for my return. But as of now I have fixed all I have discovered. |
4903e4d to
679af08
Compare
679af08 to
1ab15c3
Compare
e80831a to
0499194
Compare
|
test server has been updated (https://arched-branch-471517-f9.uc.r.appspot.com/) |
9e80046 to
ddcb7b0
Compare
RaspberryPiFoundation/blockly@75007a0#diff-bb0de0069443e7577f62ffa96e6b51c5fafd06703f50c3eed6eb616ac87467bfL118-L124 https://developers.google.com/blockly/reference/js/blockly.blocksvg_class.setwarningtext_1_method.md setWarningText checks for warning icon https://github.com/google/blockly/blob/blockly-v11.2.2/core/block_svg.ts#L972
7dda2ea to
0c383a1
Compare
| 'zoom': {'controls': true, 'wheel': true, 'scaleSpeed': 1.1, 'maxScale': 3, 'minScale': 0.1}, | ||
| plugins: { | ||
| blockDragger: MultiselectBlockDragger, | ||
| blockDragger: ScrollBlockDragger, |
There was a problem hiding this comment.
@mjgallag I'm worried that this changes the semantics of dragging. How do we end up allowing dragging multiple blocks if they are selected?
There was a problem hiding this comment.
The multiselect plugin no longer needs a custom block dragger. It was rewritten for Blockly v11 to use the IDraggable interface. MultiselectDraggable wraps selected blocks into a single draggable instead of extending BlockDragger. So ScrollBlockDragger in the dragger slot is just for auto-scrolling during drag, it doesn't touch multi-select which is handled at the draggable level.
There's a Playwright e2e test ("dragging moves selected blocks together") that covers this.
| return original(png); | ||
| }; | ||
| }); | ||
| Blockly.exportBlockAsPng(block); |
There was a problem hiding this comment.
Note that at least on my machine, this triggers the PNG to show up in my Downloads folder. We probably don't want a test that leaks files outside of the build directory.
| <script src="block.js"></script> | ||
| <script src="text.js"></script> | ||
| <script src="../../../../../build/javascript/BlocklyEvalTest.js"></script> | ||
| <script src="../../../../../build/javascript/xml_serialization_tests.js"></script> |
There was a problem hiding this comment.
Note that we should probably also add the other tests here to be able to use mocha in the browser. Also, two tests fail due to the lack of FieldColour.
There was a problem hiding this comment.
Ah, had I run the tests in Mocha I would have noticed the blocks download! #3805
ewpatton
left a comment
There was a problem hiding this comment.
I'm approving this so we can move the release forward. Most of my comments are related to the test apparatus, which we can solve in a separate PR.


General items:
ant testspasses on my machineIf your code changes how something works on the device (i.e., it affects the companion):
ucrucras the baseFurther, if you've changed the blocks language or another user-facing designer/blocks API (added a SimpleProperty, etc.):
For all other changes:
mastermasteras the baseWhat does this PR accomplish?
Resolves #3434, Resolves #3529, Resolves #3530, Resolves #3531, Resolves #3532