Blockly Added Capability #486
Conversation
…ch should fix tooltips and mutator callout origins for Blockly).
| var negateOverlap = 35; | ||
| // SJF: Overlap not present in Blockly with categories enabled | ||
|
|
||
| // var negateOverlap = 35; | ||
| Blockly.updateToolbox( xml ); |
There was a problem hiding this comment.
If the only thing that's executing in this function is Blockly.updateToolbox( xml ), does it make more sense to comment out the entire cleanUpdateToolbox function, and replace it with updateToolbox?
There was a problem hiding this comment.
Leaving it incase we switch back to using basic blockly with no categories or build our own categories or something. Good point though :)
There was a problem hiding this comment.
Hrm. Might be better to remove the cruft, and put in a comment saying something like "If you ever want to change back to the simple UI, look in github back to thus-and-such a date/branch."
On a related note, though... are level designers going to have the option to switch between the two UIs, or do we have to pick one and stick with it?
There was a problem hiding this comment.
Unfortunately you need to pick one and stick with it for the whole "game" - once it loads it must stay in a certain configuration. Haven't explored what reloading might mean/entail but I suspect it isn't pretty.
…apability-support Blockly Added Capability
...brings back the trashcan, removes some flyout offsets since we're using advanced categories and should fix tooltips and mutator callout origins for Blockly.
@kadst43 @BrettASwift @nmarshak1337