-
Notifications
You must be signed in to change notification settings - Fork 625
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
refactor!: Update and modernize the continuous toolbox plugin. #2468
base: rc/v12.0.0
Are you sure you want to change the base?
Conversation
Might it be worth creating a v12 branch in this repo (if there is not already one) and retargeting this PR to it? |
Done! |
This also fixes #2471. |
And also fixes google/blockly#8543 |
The basics
The details
Resolves
Proposed Changes
This PR updates the continuous toolbox plugin for compatibility with the revised flyout APIs introduced in Blockly v12. It also converts the plugin to Typescript.
Additionally, it fixes several existing issues:
Blockly.getMainWorkspace().getFlyout().setAutoClose(true)
to enable this behavior.This PR should not be merged until Blockly v12 is released, as it depends upon APIs added in that version.