Fix toolbox types (#4355)

core/toolbox/category.js

@@ -121,14 +121,14 @@ Blockly.utils.object.inherits(Blockly.ToolboxCategory, Blockly.ToolboxItem);
 /**
  * All the css class names that are used to create a category.
  * @typedef {{
- *            container:?string,
- *            row:?string,
- *            rowcontentcontainer:?string,
- *            icon:?string,
- *            label:?string,
- *            selected:?string,
- *            openicon:?string,
- *            closedicon:?string
+ *            container:(string|undefined),
+ *            row:(string|undefined),
+ *            rowcontentcontainer:(string|undefined),
+ *            icon:(string|undefined),
+ *            label:(string|undefined),
+ *            selected:(string|undefined),
+ *            openicon:(string|undefined),
+ *            closedicon:(string|undefined)
  *          }}
  */
 Blockly.ToolboxCategory.CssConfig;

core/toolbox/separator.js

@@ -53,7 +53,7 @@ Blockly.utils.object.inherits(Blockly.ToolboxSeparator, Blockly.ToolboxItem);
 /**
  * All the css class names that are used to create a separator.
  * @typedef {{
- *            container:?string
+ *            container:(string|undefined)
  *          }}
  */
 Blockly.ToolboxSeparator.CssConfig;

core/toolbox/toolbox.js

@@ -386,7 +386,7 @@ Blockly.Toolbox.prototype.renderContents_ = function(toolboxDef) {
 
 /**
  * Creates and renders the toolbox item.
- * @param {Blockly.utils.toolbox.ToolboxItemInfo} toolboxItemDef Any information
+ * @param {!Blockly.utils.toolbox.ToolboxItemInfo} toolboxItemDef Any information
  *    that can be used to create an item in the toolbox.
  * @param {!DocumentFragment} fragment The document fragment to add the child
  *     toolbox elements to.

core/utils/toolbox.js

@@ -20,10 +20,10 @@ goog.requireType('Blockly.ToolboxSeparator');
  * The information needed to create a block in the toolbox.
  * @typedef {{
  *            kind:string,
- *            blockxml:(?string|Node),
- *            type: ?string,
- *            gap: (?string|?number),
- *            disabled: (?string|?boolean)
+ *            blockxml:(string|!Node|undefined),
+ *            type:(string|undefined),
+ *            gap:(string|number|undefined),
+ *            disabled: (string|boolean|undefined)
  *          }}
  */
 Blockly.utils.toolbox.BlockInfo;
@@ -32,9 +32,9 @@ Blockly.utils.toolbox.BlockInfo;
  * The information needed to create a separator in the toolbox.
  * @typedef {{
  *            kind:string,
- *            id:?string,
- *            gap:?number,
- *            cssconfig:?Blockly.ToolboxSeparator.CssConfig
+ *            id:(string|undefined),
+ *            gap:(number|undefined),
+ *            cssconfig:(!Blockly.ToolboxSeparator.CssConfig|undefined)
  *          }}
  */
 Blockly.utils.toolbox.SeparatorInfo;
@@ -53,8 +53,8 @@ Blockly.utils.toolbox.ButtonInfo;
  * The information needed to create a label in the toolbox.
  * @typedef {{
  *            kind:string,
- *            id:?string,
- *            text:string
+ *            text:string,
+ *            id:(string|undefined)
  *          }}
  */
 Blockly.utils.toolbox.LabelInfo;
@@ -71,12 +71,12 @@ Blockly.utils.toolbox.ButtonOrLabelInfo;
  * @typedef {{
  *            kind:string,
  *            name:string,
- *            id:?string,
- *            categorystyle:?string,
- *            colour:?string,
- *            cssconfig:?Blockly.ToolboxCategory.CssConfig,
- *            contents:!Array<Blockly.utils.toolbox.ToolboxItemInfo>,
- *            hidden:?string
+ *            contents:!Array<!Blockly.utils.toolbox.ToolboxItemInfo>,
+ *            id:(string|undefined),
+ *            categorystyle:(string|undefined),
+ *            colour:(string|undefined),
+ *            cssconfig:(!Blockly.ToolboxCategory.CssConfig|undefined),
+ *            hidden:(string|undefined)
  *          }}
  */
 Blockly.utils.toolbox.StaticCategoryInfo;
@@ -86,11 +86,11 @@ Blockly.utils.toolbox.StaticCategoryInfo;
  * @typedef {{
  *            kind:string,
  *            custom:string,
- *            id:?string,
- *            categorystyle:?string,
- *            colour:?string,
- *            cssconfig:?Blockly.ToolboxCategory.CssConfig,
- *            hidden:?string
+ *            id:(string|undefined),
+ *            categorystyle:(string|undefined),
+ *            colour:(string|undefined),
+ *            cssconfig:(!Blockly.ToolboxCategory.CssConfig|undefined),
+ *            hidden:(string|undefined)
  *          }}
  */
 Blockly.utils.toolbox.DynamicCategoryInfo;
@@ -122,7 +122,8 @@ Blockly.utils.toolbox.FlyoutItemInfo;
 /**
  * The JSON definition of a toolbox.
  * @typedef {{
- *            contents:!Array<Blockly.utils.toolbox.ToolboxItemInfo>
+ *            kind:(string|undefined),
+ *            contents:!Array<!Blockly.utils.toolbox.ToolboxItemInfo>
  *          }}
  */
 Blockly.utils.toolbox.ToolboxInfo;