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

Several Blockly fixes #9939

Merged
merged 5 commits into from
Mar 26, 2024
Merged

Several Blockly fixes #9939

merged 5 commits into from
Mar 26, 2024

Conversation

riknoll
Copy link
Member

@riknoll riknoll commented Mar 26, 2024

Fixes microsoft/pxt-microbit#5564
Fixes microsoft/pxt-microbit#5562
Fixes microsoft/pxt-microbit#5555
Fixes microsoft/pxt-microbit#5554
Fixes microsoft/pxt-microbit#5553

Several fixes of new-blockly regressions:

  • quotes are now red again
  • text blocks now have default parameters again (this was actually a bug in our old Blockly code, but I brought the default values back to mimic the old buggy behavior)
  • adds a light contrasting outline for dark shadow blocks
  • removes references to inDebugWorkspace, which is a method that no longer exists
  • restores the old behavior of the slider field where it grabbed the track color from the parent block

I want to add a new test runner for our toolbox block XML generating code, but I'll do that in a separate PR

@riknoll riknoll requested a review from a team March 26, 2024 21:31
@riknoll riknoll merged commit 5c1b517 into master Mar 26, 2024
6 checks passed
@riknoll riknoll deleted the dev/riknoll/mb-block-fixes branch March 26, 2024 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants