jenkins-bot has submitted this change and it was merged.

Change subject: Doc: Replace "@property @type {Type}" with "@property {Type}".
......................................................................


Doc: Replace "@property @type {Type}" with "@property {Type}".

Also removed a few redundant headings in class documentation
comments. There is already an @class and it looks a bit odd  in
the generated pages:
 <h2>TextString</h2>
 <p>TextString</p>
 <p>This class provides a ...</p>

Change-Id: Ie311c6993ed02e79272dbde71f6a1bc252ef3037
---
M modules/unicodejs/unicodejs.textstring.js
M modules/ve/ce/ve.ce.BranchNode.js
M modules/ve/ce/ve.ce.Surface.js
M modules/ve/ui/dialogs/ve.ui.ContentDialog.js
M modules/ve/ui/dialogs/ve.ui.MetaDialog.js
M modules/ve/ui/tools/dropdowns/ve.ui.FormatDropdownTool.js
M modules/ve/ui/tools/ve.ui.AnnotationButtonTool.js
M modules/ve/ui/tools/ve.ui.ButtonTool.js
M modules/ve/ui/tools/ve.ui.IndentationButtonTool.js
M modules/ve/ui/tools/ve.ui.InspectorButtonTool.js
M modules/ve/ui/tools/ve.ui.ListButtonTool.js
M modules/ve/ui/ve.ui.Element.js
M modules/ve/ui/ve.ui.Inspector.js
M modules/ve/ui/ve.ui.Tool.js
M modules/ve/ui/ve.ui.Window.js
M modules/ve/ui/widgets/ve.ui.InputWidget.js
M modules/ve/ve.Surface.js
17 files changed, 21 insertions(+), 46 deletions(-)

Approvals:
  Trevor Parscal: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/unicodejs/unicodejs.textstring.js 
b/modules/unicodejs/unicodejs.textstring.js
index 4bef395..b8ddf8e 100644
--- a/modules/unicodejs/unicodejs.textstring.js
+++ b/modules/unicodejs/unicodejs.textstring.js
@@ -6,8 +6,6 @@
  */
 
 /**
- * TextString
- *
  * This class provides a simple interface to fetching plain text
  * from a data source. The base class reads data from a string, but
  * an extended class could provide access to a more complex structure,
diff --git a/modules/ve/ce/ve.ce.BranchNode.js 
b/modules/ve/ce/ve.ce.BranchNode.js
index 7838886..811dd46 100644
--- a/modules/ve/ce/ve.ce.BranchNode.js
+++ b/modules/ve/ce/ve.ce.BranchNode.js
@@ -63,8 +63,7 @@
  * Inline slug template.
  *
  * @static
- * @property
- * @type {jQuery}
+ * @property {jQuery}
  */
 ve.ce.BranchNode.$inlineSlugTemplate = $( '<span>' )
        .addClass( 've-ce-slug' )
@@ -74,8 +73,7 @@
  * Block slug template.
  *
  * @static
- * @property
- * @type {jQuery}
+ * @property {jQuery}
  */
 ve.ce.BranchNode.$blockSlugTemplate =
        ve.ce.BranchNode.$inlineSlugTemplate.clone().addClass( 
've-ce-slugBlock' );
diff --git a/modules/ve/ce/ve.ce.Surface.js b/modules/ve/ce/ve.ce.Surface.js
index 53b2471..5d95f1a 100644
--- a/modules/ve/ce/ve.ce.Surface.js
+++ b/modules/ve/ce/ve.ce.Surface.js
@@ -92,8 +92,7 @@
  * Phantom element template.
  *
  * @static
- * @property
- * @type {jQuery}
+ * @property {jQuery}
  */
 ve.ce.Surface.static.$phantomTemplate = $( '<div>' )
        .addClass( 've-ce-surface-phantom' )
@@ -106,8 +105,7 @@
  * Pattern matching "normal" characters which we can let the browser handle 
natively.
  *
  * @static
- * @property
- * @type {RegExp}
+ * @property {RegExp}
  */
 ve.ce.Surface.static.textPattern = new RegExp(
        
'[a-zA-Z\\-_’\'‘ÆÐƎƏƐƔIJŊŒẞÞǷȜæðǝəɛɣijŋœĸſßþƿȝĄƁÇĐƊĘĦĮƘŁØƠŞȘŢȚŦŲƯY̨Ƴąɓçđɗęħįƙłøơşșţțŧųưy̨ƴÁÀÂÄ'
 +
diff --git a/modules/ve/ui/dialogs/ve.ui.ContentDialog.js 
b/modules/ve/ui/dialogs/ve.ui.ContentDialog.js
index be2f506..1a72c32 100644
--- a/modules/ve/ui/dialogs/ve.ui.ContentDialog.js
+++ b/modules/ve/ui/dialogs/ve.ui.ContentDialog.js
@@ -30,8 +30,7 @@
  * Localized message for dialog title.
  *
  * @static
- * @property
- * @type {string}
+ * @property {string}
  */
 ve.ui.ContentDialog.static.titleMessage = 'visualeditor-dialog-content-title';
 
diff --git a/modules/ve/ui/dialogs/ve.ui.MetaDialog.js 
b/modules/ve/ui/dialogs/ve.ui.MetaDialog.js
index 4d3a063..bb88c94 100644
--- a/modules/ve/ui/dialogs/ve.ui.MetaDialog.js
+++ b/modules/ve/ui/dialogs/ve.ui.MetaDialog.js
@@ -30,8 +30,7 @@
  * Localized message for dialog title.
  *
  * @static
- * @property
- * @type {string}
+ * @property {string}
  */
 ve.ui.MetaDialog.static.titleMessage = 'visualeditor-dialog-meta-title';
 
diff --git a/modules/ve/ui/tools/dropdowns/ve.ui.FormatDropdownTool.js 
b/modules/ve/ui/tools/dropdowns/ve.ui.FormatDropdownTool.js
index 843f76e..c1769d7 100644
--- a/modules/ve/ui/tools/dropdowns/ve.ui.FormatDropdownTool.js
+++ b/modules/ve/ui/tools/dropdowns/ve.ui.FormatDropdownTool.js
@@ -45,8 +45,7 @@
  * Options given to ve.ui.DropdownTool.
  *
  * @static
- * @property
- * @type {Object[]}
+ * @property {Object[]}
  */
 ve.ui.FormatDropdownTool.static.items = [
        {
diff --git a/modules/ve/ui/tools/ve.ui.AnnotationButtonTool.js 
b/modules/ve/ui/tools/ve.ui.AnnotationButtonTool.js
index b7e2e70..cdc0245 100644
--- a/modules/ve/ui/tools/ve.ui.AnnotationButtonTool.js
+++ b/modules/ve/ui/tools/ve.ui.AnnotationButtonTool.js
@@ -32,8 +32,7 @@
  *
  * @abstract
  * @static
- * @property
- * @type {Object}
+ * @property {Object}
  * @inheritable
  */
 ve.ui.AnnotationButtonTool.static.annotation = { 'name': '' };
diff --git a/modules/ve/ui/tools/ve.ui.ButtonTool.js 
b/modules/ve/ui/tools/ve.ui.ButtonTool.js
index 9449a2e..a4ce91b 100644
--- a/modules/ve/ui/tools/ve.ui.ButtonTool.js
+++ b/modules/ve/ui/tools/ve.ui.ButtonTool.js
@@ -63,8 +63,7 @@
  *
  * @abstract
  * @static
- * @property
- * @type {string|Object}
+ * @property {string|Object}
  * @inheritable
  */
 ve.ui.ButtonTool.static.icon = '';
diff --git a/modules/ve/ui/tools/ve.ui.IndentationButtonTool.js 
b/modules/ve/ui/tools/ve.ui.IndentationButtonTool.js
index 05cbd81..765de69 100644
--- a/modules/ve/ui/tools/ve.ui.IndentationButtonTool.js
+++ b/modules/ve/ui/tools/ve.ui.IndentationButtonTool.js
@@ -31,8 +31,7 @@
  *
  * @abstract
  * @static
- * @property
- * @type {string}
+ * @property {string}
  * @inheritable
  */
 ve.ui.IndentationButtonTool.static.method = '';
diff --git a/modules/ve/ui/tools/ve.ui.InspectorButtonTool.js 
b/modules/ve/ui/tools/ve.ui.InspectorButtonTool.js
index 6b6ab13..c67e7c1 100644
--- a/modules/ve/ui/tools/ve.ui.InspectorButtonTool.js
+++ b/modules/ve/ui/tools/ve.ui.InspectorButtonTool.js
@@ -31,8 +31,7 @@
  *
  * @abstract
  * @static
- * @property
- * @type {string}
+ * @property {string}
  * @inheritable
  */
 ve.ui.InspectorButtonTool.static.inspector = '';
diff --git a/modules/ve/ui/tools/ve.ui.ListButtonTool.js 
b/modules/ve/ui/tools/ve.ui.ListButtonTool.js
index 5921065..954c1ed 100644
--- a/modules/ve/ui/tools/ve.ui.ListButtonTool.js
+++ b/modules/ve/ui/tools/ve.ui.ListButtonTool.js
@@ -29,8 +29,7 @@
  *
  * @abstract
  * @static
- * @property
- * @type {string}
+ * @property {string}
  * @inheritable
  */
 ve.ui.ListButtonTool.static.style = '';
diff --git a/modules/ve/ui/ve.ui.Element.js b/modules/ve/ui/ve.ui.Element.js
index 3d01262..f387c93 100644
--- a/modules/ve/ui/ve.ui.Element.js
+++ b/modules/ve/ui/ve.ui.Element.js
@@ -37,8 +37,7 @@
  * HTML element name.
  *
  * @static
- * @property
- * @type {string}
+ * @property {string}
  * @inheritable
  */
 ve.ui.Element.static.tagName = 'div';
diff --git a/modules/ve/ui/ve.ui.Inspector.js b/modules/ve/ui/ve.ui.Inspector.js
index 3b7a430..82ed02b 100644
--- a/modules/ve/ui/ve.ui.Inspector.js
+++ b/modules/ve/ui/ve.ui.Inspector.js
@@ -35,8 +35,7 @@
  * Pattern to use when matching against annotation type strings.
  *
  * @static
- * @property
- * @type {RegExp}
+ * @property {RegExp}
  */
 ve.ui.Inspector.static.typePattern = new RegExp();
 
diff --git a/modules/ve/ui/ve.ui.Tool.js b/modules/ve/ui/ve.ui.Tool.js
index a0fc42c..958fd8a 100644
--- a/modules/ve/ui/ve.ui.Tool.js
+++ b/modules/ve/ui/ve.ui.Tool.js
@@ -47,8 +47,7 @@
  *
  * @abstract
  * @static
- * @property
- * @type {string}
+ * @property {string}
  */
 ve.ui.Tool.static.name = '';
 
@@ -59,8 +58,7 @@
  *
  * @abstract
  * @static
- * @property
- * @type {string}
+ * @property {string}
  */
 ve.ui.Tool.static.cssName = null;
 
@@ -69,8 +67,7 @@
  *
  * @abstract
  * @static
- * @property
- * @type {string}
+ * @property {string}
  */
 ve.ui.Tool.static.titleMessage = null;
 
diff --git a/modules/ve/ui/ve.ui.Window.js b/modules/ve/ui/ve.ui.Window.js
index 9692362..9d5fe5e 100644
--- a/modules/ve/ui/ve.ui.Window.js
+++ b/modules/ve/ui/ve.ui.Window.js
@@ -86,8 +86,7 @@
  * @see ve.ui.Frame
  *
  * @static
- * @property
- * @type {Object}
+ * @property {Object}
  */
 ve.ui.Window.static.stylesheets = [
        've.ui.Frame.css',
@@ -101,8 +100,7 @@
  * Symbolic name of icon.
  *
  * @static
- * @property
- * @type {string}
+ * @property {string}
  */
 ve.ui.Window.static.icon = 'window';
 
@@ -110,8 +108,7 @@
  * Localized message for title.
  *
  * @static
- * @property
- * @type {string}
+ * @property {string}
  */
 ve.ui.Window.static.titleMessage = null;
 
diff --git a/modules/ve/ui/widgets/ve.ui.InputWidget.js 
b/modules/ve/ui/widgets/ve.ui.InputWidget.js
index 8601777..85d949e 100644
--- a/modules/ve/ui/widgets/ve.ui.InputWidget.js
+++ b/modules/ve/ui/widgets/ve.ui.InputWidget.js
@@ -59,8 +59,7 @@
  * HTML input type.
  *
  * @static
- * @property
- * @type {string}
+ * @property {string}
  * @inheritable
  */
 ve.ui.InputWidget.static.inputType = '';
diff --git a/modules/ve/ve.Surface.js b/modules/ve/ve.Surface.js
index b87dbe0..783983e 100644
--- a/modules/ve/ve.Surface.js
+++ b/modules/ve/ve.Surface.js
@@ -6,8 +6,6 @@
  */
 
 /**
- * Surface.
- *
  * A surface is a top-level object which contains both a surface model and a 
surface view.
  *
  *     @example

-- 
To view, visit https://gerrit.wikimedia.org/r/56516
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie311c6993ed02e79272dbde71f6a1bc252ef3037
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Krinkle <[email protected]>
Gerrit-Reviewer: Trevor Parscal <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to