Cscott has uploaded a new change for review.
https://gerrit.wikimedia.org/r/216139
Change subject: Stop using deprecated aliases for OO.ui.mixin.*
......................................................................
Stop using deprecated aliases for OO.ui.mixin.*
These classes were renamed after OOui v0.11.3.
Update the jsduck categories to match the latest OOui categories
at the same time (to reflect the new mixin category).
Change-Id: Ie932eb3bf5e65866ae4ba3bf2317facb1e740d3b
---
M .jsduck/categories.json
M modules/ve-mw/ce/nodes/ve.ce.MWAlienExtensionNode.js
M modules/ve-mw/ce/nodes/ve.ce.MWTransclusionNode.js
M modules/ve-mw/ui/widgets/ve.ui.MWCategoryInputWidget.js
M modules/ve-mw/ui/widgets/ve.ui.MWCategoryItemWidget.js
M modules/ve-mw/ui/widgets/ve.ui.MWCategoryWidget.js
M modules/ve-mw/ui/widgets/ve.ui.MWLinkTargetInputWidget.js
M modules/ve-mw/ui/widgets/ve.ui.MWMediaInfoFieldWidget.js
M modules/ve-mw/ui/widgets/ve.ui.MWTocItemWidget.js
9 files changed, 34 insertions(+), 98 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor
refs/changes/39/216139/1
diff --git a/.jsduck/categories.json b/.jsduck/categories.json
index 3933327..60c34ee 100644
--- a/.jsduck/categories.json
+++ b/.jsduck/categories.json
@@ -168,75 +168,10 @@
]
},
{
- "name": "OOJS UI",
- "groups": [
- {
- "name": "General",
- "classes": [
- "OO.ui",
- "OO.ui.ActionSet",
- "OO.ui.Element",
- "OO.ui.Layout",
- "OO.ui.Widget",
- "OO.ui.Window",
- "OO.ui.Dialog",
- "OO.ui.WindowManager",
- "OO.ui.Error",
- "OO.ui.Process",
- "OO.ui.HtmlSnippet"
- ]
- },
- {
- "name": "Elements",
- "classes": [
- "OO.ui.*Element"
- ]
- },
- {
- "name": "Tools",
- "classes": [
- "OO.ui.*Tool",
- "OO.ui.Tool*Factory"
- ]
- },
- {
- "name": "Toolbar and tool groups",
- "classes": [
- "OO.ui.Toolbar",
- "OO.ui.*ToolGroup"
- ]
- },
- {
- "name": "Dialogs",
- "classes": [
- "OO.ui.*Dialog"
- ]
- },
- {
- "name": "Layouts",
- "classes": [
- "OO.ui.*Layout"
- ]
- },
- {
- "name": "Widgets",
- "classes": [
- "OO.ui.*Widget"
- ]
- },
- {
- "name": "Themes",
- "classes": [
- "OO.ui.*Theme"
- ]
- }
- ]
- },
- {
"name": "Upstream",
"groups": [
{
- "name": "OOJS",
+ "name": "OOjs",
"classes": [
"OO",
"OO.EventEmitter",
diff --git a/modules/ve-mw/ce/nodes/ve.ce.MWAlienExtensionNode.js
b/modules/ve-mw/ce/nodes/ve.ce.MWAlienExtensionNode.js
index 9eb7137..22b1b91 100644
--- a/modules/ve-mw/ce/nodes/ve.ce.MWAlienExtensionNode.js
+++ b/modules/ve-mw/ce/nodes/ve.ce.MWAlienExtensionNode.js
@@ -10,14 +10,14 @@
*
* @class
* @abstract
- * @mixins OO.ui.IconElement
+ * @mixins OO.ui.mixin.IconElement
*
* @constructor
* @param {Object} [config] Configuration options
*/
ve.ce.MWAlienExtensionNode = function VeCeMWAlienExtensionNode( config ) {
// Mixin constructors
- OO.ui.IconElement.call( this, config );
+ OO.ui.mixin.IconElement.call( this, config );
// Events
this.connect( this, { setup: 'onAlienSetup' } );
@@ -27,7 +27,7 @@
OO.initClass( ve.ce.MWAlienExtensionNode );
-OO.mixinClass( ve.ce.MWAlienExtensionNode, OO.ui.IconElement );
+OO.mixinClass( ve.ce.MWAlienExtensionNode, OO.ui.mixin.IconElement );
/* Static members */
diff --git a/modules/ve-mw/ce/nodes/ve.ce.MWTransclusionNode.js
b/modules/ve-mw/ce/nodes/ve.ce.MWTransclusionNode.js
index 2cb3b53..75abd5e 100644
--- a/modules/ve-mw/ce/nodes/ve.ce.MWTransclusionNode.js
+++ b/modules/ve-mw/ce/nodes/ve.ce.MWTransclusionNode.js
@@ -12,6 +12,7 @@
* @abstract
* @extends ve.ce.LeafNode
* @mixins ve.ce.FocusableNode
+ * @mixins OO.ui.mixin.IconElement
* @mixins ve.ce.GeneratedContentNode
*
* @constructor
@@ -24,7 +25,7 @@
// Mixin constructors
ve.ce.FocusableNode.call( this );
- OO.ui.IconElement.call( this, config );
+ OO.ui.mixin.IconElement.call( this, config );
ve.ce.GeneratedContentNode.call( this );
};
@@ -34,7 +35,7 @@
OO.mixinClass( ve.ce.MWTransclusionNode, ve.ce.FocusableNode );
OO.mixinClass( ve.ce.MWTransclusionNode, ve.ce.GeneratedContentNode );
-OO.mixinClass( ve.ce.MWTransclusionNode, OO.ui.IconElement );
+OO.mixinClass( ve.ce.MWTransclusionNode, OO.ui.mixin.IconElement );
/* Static Properties */
diff --git a/modules/ve-mw/ui/widgets/ve.ui.MWCategoryInputWidget.js
b/modules/ve-mw/ui/widgets/ve.ui.MWCategoryInputWidget.js
index c0bbd6a..5eee3d9 100644
--- a/modules/ve-mw/ui/widgets/ve.ui.MWCategoryInputWidget.js
+++ b/modules/ve-mw/ui/widgets/ve.ui.MWCategoryInputWidget.js
@@ -10,7 +10,7 @@
*
* @class
* @extends OO.ui.TextInputWidget
- * @mixins OO.ui.LookupElement
+ * @mixins OO.ui.mixin.LookupElement
*
* @constructor
* @param {ve.ui.MWCategoryWidget} categoryWidget
@@ -26,7 +26,7 @@
OO.ui.TextInputWidget.call( this, config );
// Mixin constructors
- OO.ui.LookupElement.call( this, config );
+ OO.ui.mixin.LookupElement.call( this, config );
// Properties
this.categoryWidget = categoryWidget;
@@ -40,7 +40,7 @@
OO.inheritClass( ve.ui.MWCategoryInputWidget, OO.ui.TextInputWidget );
-OO.mixinClass( ve.ui.MWCategoryInputWidget, OO.ui.LookupElement );
+OO.mixinClass( ve.ui.MWCategoryInputWidget, OO.ui.mixin.LookupElement );
/* Events */
diff --git a/modules/ve-mw/ui/widgets/ve.ui.MWCategoryItemWidget.js
b/modules/ve-mw/ui/widgets/ve.ui.MWCategoryItemWidget.js
index ddb5b5f..fe8e5e8 100644
--- a/modules/ve-mw/ui/widgets/ve.ui.MWCategoryItemWidget.js
+++ b/modules/ve-mw/ui/widgets/ve.ui.MWCategoryItemWidget.js
@@ -11,7 +11,7 @@
* @class
* @abstract
* @extends OO.ui.ButtonWidget
- * @mixins OO.ui.DraggableElement
+ * @mixins OO.ui.mixin.DraggableElement
*
* @constructor
* @param {Object} [config] Configuration options
@@ -28,7 +28,7 @@
OO.ui.ButtonWidget.call( this, config );
// Mixin constructors
- OO.ui.DraggableElement.call( this, config );
+ OO.ui.mixin.DraggableElement.call( this, config );
// Properties
this.name = config.item.name;
@@ -61,7 +61,7 @@
OO.inheritClass( ve.ui.MWCategoryItemWidget, OO.ui.ButtonWidget );
-OO.mixinClass( ve.ui.MWCategoryItemWidget, OO.ui.DraggableElement );
+OO.mixinClass( ve.ui.MWCategoryItemWidget, OO.ui.mixin.DraggableElement );
/* Events */
diff --git a/modules/ve-mw/ui/widgets/ve.ui.MWCategoryWidget.js
b/modules/ve-mw/ui/widgets/ve.ui.MWCategoryWidget.js
index 99e7f32..56af3f0 100644
--- a/modules/ve-mw/ui/widgets/ve.ui.MWCategoryWidget.js
+++ b/modules/ve-mw/ui/widgets/ve.ui.MWCategoryWidget.js
@@ -11,8 +11,8 @@
* @class
* @abstract
* @extends OO.ui.Widget
- * @mixins OO.ui.GroupElement
- * @mixins OO.ui.DraggableGroupElement
+ * @mixins OO.ui.mixin.GroupElement
+ * @mixins OO.ui.mixin.DraggableGroupElement
*
* @constructor
* @param {Object} [config] Configuration options
@@ -26,8 +26,8 @@
OO.ui.Widget.call( this, config );
// Mixin constructors
- OO.ui.GroupElement.call( this, config );
- OO.ui.DraggableGroupElement.call( this, $.extend( {}, config, {
orientation: 'horizontal' } ) );
+ OO.ui.mixin.GroupElement.call( this, config );
+ OO.ui.mixin.DraggableGroupElement.call( this, $.extend( {}, config, {
orientation: 'horizontal' } ) );
// Properties
this.categories = {};
@@ -62,8 +62,8 @@
OO.inheritClass( ve.ui.MWCategoryWidget, OO.ui.Widget );
-OO.mixinClass( ve.ui.MWCategoryWidget, OO.ui.GroupElement );
-OO.mixinClass( ve.ui.MWCategoryWidget, OO.ui.DraggableGroupElement );
+OO.mixinClass( ve.ui.MWCategoryWidget, OO.ui.mixin.GroupElement );
+OO.mixinClass( ve.ui.MWCategoryWidget, OO.ui.mixin.DraggableGroupElement );
/* Events */
@@ -183,7 +183,7 @@
* @inheritdoc
*/
ve.ui.MWCategoryWidget.prototype.clearItems = function () {
- OO.ui.GroupElement.prototype.clearItems.call( this );
+ OO.ui.mixin.GroupElement.prototype.clearItems.call( this );
this.categories = {};
};
@@ -350,7 +350,7 @@
categoryItems.push( categoryItem );
}
- OO.ui.DraggableGroupElement.prototype.addItems.call(
categoryWidget, categoryItems, index );
+ OO.ui.mixin.DraggableGroupElement.prototype.addItems.call(
categoryWidget, categoryItems, index );
categoryWidget.fitInput();
} );
@@ -371,7 +371,7 @@
}
}
- OO.ui.DraggableGroupElement.prototype.removeItems.call( this, items );
+ OO.ui.mixin.DraggableGroupElement.prototype.removeItems.call( this,
items );
this.fitInput();
};
diff --git a/modules/ve-mw/ui/widgets/ve.ui.MWLinkTargetInputWidget.js
b/modules/ve-mw/ui/widgets/ve.ui.MWLinkTargetInputWidget.js
index 57c716e..85f2886 100644
--- a/modules/ve-mw/ui/widgets/ve.ui.MWLinkTargetInputWidget.js
+++ b/modules/ve-mw/ui/widgets/ve.ui.MWLinkTargetInputWidget.js
@@ -10,7 +10,7 @@
*
* @class
* @extends OO.ui.TextInputWidget
- * @mixins OO.ui.LookupElement
+ * @mixins OO.ui.mixin.LookupElement
*
* @constructor
* @param {Object} [config] Configuration options
@@ -25,7 +25,7 @@
ve.ui.MWLinkTargetInputWidget.super.call( this, config );
// Mixin constructors
- OO.ui.LookupElement.call( this, config );
+ OO.ui.mixin.LookupElement.call( this, config );
// Initialization
this.$element.addClass( 've-ui-mwLinkTargetInputWidget' );
@@ -53,7 +53,7 @@
OO.inheritClass( ve.ui.MWLinkTargetInputWidget, OO.ui.TextInputWidget );
-OO.mixinClass( ve.ui.MWLinkTargetInputWidget, OO.ui.LookupElement );
+OO.mixinClass( ve.ui.MWLinkTargetInputWidget, OO.ui.mixin.LookupElement );
/* Methods */
diff --git a/modules/ve-mw/ui/widgets/ve.ui.MWMediaInfoFieldWidget.js
b/modules/ve-mw/ui/widgets/ve.ui.MWMediaInfoFieldWidget.js
index d88a643..327b063 100644
--- a/modules/ve-mw/ui/widgets/ve.ui.MWMediaInfoFieldWidget.js
+++ b/modules/ve-mw/ui/widgets/ve.ui.MWMediaInfoFieldWidget.js
@@ -10,8 +10,8 @@
*
* @class
* @extends OO.ui.Widget
- * @mixins OO.ui.IconElement
- * @mixins OO.ui.TitledElement
+ * @mixins OO.ui.mixin.IconElement
+ * @mixins OO.ui.mixin.TitledElement
*
* @constructor
* @param {Object} [config] Configuration options
@@ -31,8 +31,8 @@
ve.ui.MWMediaInfoFieldWidget.super.call( this, config );
// Mixin constructors
- OO.ui.IconElement.call( this, config );
- OO.ui.LabelElement.call( this, $.extend( { $label: $( '<div>' ) },
config ) );
+ OO.ui.mixin.IconElement.call( this, config );
+ OO.ui.mixin.LabelElement.call( this, $.extend( { $label: $( '<div>' )
}, config ) );
this.$text = $( '<div>' )
.addClass( 've-ui-mwMediaInfoFieldWidget-text' );
@@ -98,8 +98,8 @@
/* Setup */
OO.inheritClass( ve.ui.MWMediaInfoFieldWidget, OO.ui.Widget );
-OO.mixinClass( ve.ui.MWMediaInfoFieldWidget, OO.ui.IconElement );
-OO.mixinClass( ve.ui.MWMediaInfoFieldWidget, OO.ui.LabelElement );
+OO.mixinClass( ve.ui.MWMediaInfoFieldWidget, OO.ui.mixin.IconElement );
+OO.mixinClass( ve.ui.MWMediaInfoFieldWidget, OO.ui.mixin.LabelElement );
/* Static Properties */
diff --git a/modules/ve-mw/ui/widgets/ve.ui.MWTocItemWidget.js
b/modules/ve-mw/ui/widgets/ve.ui.MWTocItemWidget.js
index 5ea2eb9..62e5554 100644
--- a/modules/ve-mw/ui/widgets/ve.ui.MWTocItemWidget.js
+++ b/modules/ve-mw/ui/widgets/ve.ui.MWTocItemWidget.js
@@ -10,7 +10,7 @@
*
* @class
* @extends OO.ui.Widget
- * @mixins OO.ui.GroupElement
+ * @mixins OO.ui.mixin.GroupElement
*
* @constructor
* @param {Object} config TOC Item configuration
@@ -26,7 +26,7 @@
OO.ui.Widget.call( this, config );
// Mixin Constructor
- OO.ui.GroupElement.call( this, $.extend( {}, config, { $group: $(
'<ul>' ) } ) );
+ OO.ui.mixin.GroupElement.call( this, $.extend( {}, config, { $group: $(
'<ul>' ) } ) );
config = config || {};
@@ -59,7 +59,7 @@
OO.inheritClass( ve.ui.MWTocItemWidget, OO.ui.Widget );
-OO.mixinClass( ve.ui.MWTocItemWidget, OO.ui.GroupElement );
+OO.mixinClass( ve.ui.MWTocItemWidget, OO.ui.mixin.GroupElement );
/* Static Properties */
--
To view, visit https://gerrit.wikimedia.org/r/216139
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie932eb3bf5e65866ae4ba3bf2317facb1e740d3b
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Cscott <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
