Fix jsdoc errors and warnings (#4309)

core/contextmenu_registry.js

@@ -52,7 +52,7 @@ Blockly.ContextMenuRegistry.ScopeType = {
  * displayText functions that depend on this information.
  * @typedef {{
  *    block: (Blockly.BlockSvg|undefined),
- *    workspace: (Blockly.WorkspaceSvg|undefined),
+ *    workspace: (Blockly.WorkspaceSvg|undefined)
  * }}
  */
 Blockly.ContextMenuRegistry.Scope;
@@ -65,7 +65,7 @@ Blockly.ContextMenuRegistry.Scope;
  *    displayText: ((function(!Blockly.ContextMenuRegistry.Scope):string)|string),
  *    preconditionFn: function(!Blockly.ContextMenuRegistry.Scope):string,
  *    weight: number,
- *    id: string,
+ *    id: string
  * }}
 */
 Blockly.ContextMenuRegistry.RegistryItem;
@@ -77,7 +77,7 @@ Blockly.ContextMenuRegistry.RegistryItem;
  *    enabled: boolean,
  *    callback: function(!Blockly.ContextMenuRegistry.Scope),
  *    scope: !Blockly.ContextMenuRegistry.Scope,
- *    weight: number,
+ *    weight: number
  * }}
  */
 Blockly.ContextMenuRegistry.ContextMenuOption;

core/options.js

@@ -201,7 +201,7 @@ Blockly.BlocklyOptions = function() {};
  *     colour: string,
  *     length: number,
  *     snap: boolean,
- *     spacing: number,
+ *     spacing: number
  * }}
  */
 Blockly.Options.GridOptions;
@@ -211,7 +211,7 @@ Blockly.Options.GridOptions;
  * @typedef {{
  *     drag: boolean,
  *     scrollbars: boolean,
- *     wheel: boolean,
+ *     wheel: boolean
  * }}
  */
 Blockly.Options.MoveOptions;
@@ -225,7 +225,7 @@ Blockly.Options.MoveOptions;
  *     pinch: boolean,
  *     scaleSpeed: number,
  *     startScale: number,
- *     wheel: boolean,
+ *     wheel: boolean
  * }}
  */
 Blockly.Options.ZoomOptions;

core/toolbox/category.js

@@ -127,7 +127,7 @@ Blockly.utils.object.inherits(Blockly.ToolboxCategory, Blockly.ToolboxItem);
  *            label:?string,
  *            selected:?string,
  *            openIcon:?string,
- *            closedIcon:?string,
+ *            closedIcon:?string
  *          }}
  */
 Blockly.ToolboxCategory.CssConfig;

core/toolbox/collapsible_category.js

@@ -76,7 +76,7 @@ Blockly.utils.object.inherits(Blockly.CollapsibleToolboxCategory, Blockly.Toolbo
  *            selected:?string,
  *            openIcon:?string,
  *            closedIcon:?string,
- *            contents:?string,
+ *            contents:?string
  *          }}
  */
 Blockly.CollapsibleToolboxCategory.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
  *          }}
  */
 Blockly.ToolboxSeparator.CssConfig;

core/utils/svg.js

@@ -42,94 +42,140 @@ Blockly.utils.Svg.prototype.toString = function() {
   return this.tagName_;
 };
 
-/** @type {!Blockly.utils.Svg<!SVGAnimateElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGAnimateElement>}
+ * @package
+ */
 Blockly.utils.Svg.ANIMATE =
     new Blockly.utils.Svg('animate');
 
-/** @type {!Blockly.utils.Svg<!SVGCircleElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGCircleElement>}
+* @package
+ */
 Blockly.utils.Svg.CIRCLE =
     new Blockly.utils.Svg('circle');
 
-/** @type {!Blockly.utils.Svg<!SVGClipPathElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGClipPathElement>}
+ * @package
+ */
 Blockly.utils.Svg.CLIPPATH =
     new Blockly.utils.Svg('clipPath');
 
-/** @type {!Blockly.utils.Svg<!SVGDefsElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGDefsElement>}
+ * @package
+ */
 Blockly.utils.Svg.DEFS =
     new Blockly.utils.Svg('defs');
 
-/** @type {!Blockly.utils.Svg<!SVGFECompositeElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGFECompositeElement>}
+ * @package
+ */
 Blockly.utils.Svg.FECOMPOSITE =
     new Blockly.utils.Svg('feComposite');
 
-/** @type {!Blockly.utils.Svg<!SVGFEComponentTransferElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGFEComponentTransferElement>}
+ * @package
+ */
 Blockly.utils.Svg.FECOMPONENTTRANSFER =
     new Blockly.utils.Svg('feComponentTransfer');
 
-/** @type {!Blockly.utils.Svg<!SVGFEFloodElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGFEFloodElement>}
+ * @package
+ */
 Blockly.utils.Svg.FEFLOOD =
     new Blockly.utils.Svg('feFlood');
 
-/** @type {!Blockly.utils.Svg<!SVGFEFuncAElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGFEFuncAElement>}
+ * @package
+ */
 Blockly.utils.Svg.FEFUNCA =
     new Blockly.utils.Svg('feFuncA');
 
-/** @type {!Blockly.utils.Svg<!SVGFEGaussianBlurElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGFEGaussianBlurElement>}
+ * @package
+ */
 Blockly.utils.Svg.FEGAUSSIANBLUR =
     new Blockly.utils.Svg('feGaussianBlur');
 
-/** @type {!Blockly.utils.Svg<!SVGFEPointLightElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGFEPointLightElement>}
+ * @package
+ */
 Blockly.utils.Svg.FEPOINTLIGHT =
     new Blockly.utils.Svg('fePointLight');
 
-/** @type {!Blockly.utils.Svg<!SVGFESpecularLightingElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGFESpecularLightingElement>}
+ * @package
+ */
 Blockly.utils.Svg.FESPECULARLIGHTING =
     new Blockly.utils.Svg('feSpecularLighting');
 
-/** @type {!Blockly.utils.Svg<!SVGFilterElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGFilterElement>}
+ * @package
+ */
 Blockly.utils.Svg.FILTER =
     new Blockly.utils.Svg('filter');
 
-/** @type {!Blockly.utils.Svg<!SVGForeignObjectElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGForeignObjectElement>}
+ * @package
+ */
 Blockly.utils.Svg.FOREIGNOBJECT =
     new Blockly.utils.Svg('foreignObject');
 
-/** @type {!Blockly.utils.Svg<!SVGGElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGGElement>}
+ * @package
+ */
 Blockly.utils.Svg.G =
     new Blockly.utils.Svg('g');
 
-/** @type {!Blockly.utils.Svg<!SVGImageElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGImageElement>}
+ * @package
+ */
 Blockly.utils.Svg.IMAGE =
     new Blockly.utils.Svg('image');
 
-/** @type {!Blockly.utils.Svg<!SVGLineElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGLineElement>}
+ * @package
+ */
 Blockly.utils.Svg.LINE =
     new Blockly.utils.Svg('line');
 
-/** @type {!Blockly.utils.Svg<!SVGPathElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGPathElement>}
+ * @package
+ */
 Blockly.utils.Svg.PATH =
     new Blockly.utils.Svg('path');
 
-/** @type {!Blockly.utils.Svg<!SVGPatternElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGPatternElement>}
+ * @package
+ */
 Blockly.utils.Svg.PATTERN =
     new Blockly.utils.Svg('pattern');
 
-/** @type {!Blockly.utils.Svg<!SVGPolygonElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGPolygonElement>}
+ * @package
+ */
 Blockly.utils.Svg.POLYGON =
     new Blockly.utils.Svg('polygon');
 
-/** @type {!Blockly.utils.Svg<!SVGRectElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGRectElement>}
+ * @package
+ */
 Blockly.utils.Svg.RECT =
     new Blockly.utils.Svg('rect');
 
-/** @type {!Blockly.utils.Svg<!SVGSVGElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGSVGElement>}
+ * @package
+ */
 Blockly.utils.Svg.SVG =
     new Blockly.utils.Svg('svg');
 
-/** @type {!Blockly.utils.Svg<!SVGTextElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGTextElement>}
+ * @package
+ */
 Blockly.utils.Svg.TEXT =
     new Blockly.utils.Svg('text');
 
-/** @type {!Blockly.utils.Svg<!SVGTSpanElement>} @package */
+/** @type {!Blockly.utils.Svg<!SVGTSpanElement>}
+ * @package
+ */
 Blockly.utils.Svg.TSPAN =
     new Blockly.utils.Svg('tspan');