-
Notifications
You must be signed in to change notification settings - Fork 479
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
Disallow behavior editing if there is a toolbox with no categories #32395
Conversation
This seems like a workaround for a very specific use-case. Is there future work planned to allow levelbuilders to enable/disable the edit behavior button per level? |
I don't think so at this point? This was the decision from the curriculum/eng meeting: https://docs.google.com/document/d/1EL99K31sSv3wwsfc5bNqtChh870oF3GepenWxyPm9Cg/edit#bookmark=id.swq3i21lhwkw |
@@ -377,7 +378,12 @@ export default { | |||
.appendTitle(fieldLabel, 'VAR') | |||
.appendTitle(Blockly.Msg.VARIABLES_GET_TAIL); | |||
|
|||
if (Blockly.useModalFunctionEditor) { | |||
let allowBehaviorEditing = Blockly.useModalFunctionEditor; | |||
if (appOptions.level.toolbox && !Blockly.hasCategories) { |
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.
Can you add a comment explaining why we do this check? (No need to wait for drone after)
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.
Left one comment. Besides that, looks good!
Description
Fix for #32252, which was reverted by #32280
Level with categorized toolbox (allows behavior editing):
s/coursef-2019/stage/3/puzzle/5
Level with no toolbox (allows behavior editing):
s/coursef-2019/stage/15/puzzle/2
Level with uncategorized toolbox (disallows behavior editing):
s/coursef-2019/stage/2/puzzle/4
Links
Testing story
Reviewer Checklist: