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
Fixes for LB page and readonly blockspace #35523
Conversation
@@ -1040,7 +1039,11 @@ exports.createJsWrapperBlockCreator = function( | |||
}); | |||
} | |||
}); | |||
if (appOptions.level.miniToolbox && !appOptions.readonlyWorkspace) { | |||
if ( | |||
!window.appOptions || |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i don't understand the addition of !window.appOptions
here -- why do we init mini-toolbox if we don't have appOptions?
(out of curiosity) are we moving away from using /* global appOptions */
in favor of window.appOptions
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is for the levelbuilder block page which doesn't have appOptions
but we do want to render the mini toolbox blocks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, right, that makes sense!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(added a comment to the code for clarity)
Relies on code-dot-org/blockly#233
Before:
After (not perfect, but at least not broken):
Also switches the default on the pointer blocks to be the long string, so that it renders properly in instructions:
Before:
After:
Links
Testing story
Reviewer Checklist: