jenkins-bot has submitted this change and it was merged. Change subject: Update OOjs UI to v0.1.0-pre (8f8896196f) ......................................................................
Update OOjs UI to v0.1.0-pre (8f8896196f) New changes: 56587a8 [BREAKING CHANGE] Rename InputWidget#sanitizeValue → #cleanUpValue 4253739 Implement radio button widgets Local changes: LinkTargetInputWidget: Update for #sanitizeValue → #cleanUpValue OOUI change Bug: 73156 Change-Id: If10927a4e595de9b6d0ad45182470e84f335683c --- M lib/oojs-ui/oojs-ui-apex.css M lib/oojs-ui/oojs-ui-apex.js M lib/oojs-ui/oojs-ui-apex.rtl.css M lib/oojs-ui/oojs-ui-apex.svg.css M lib/oojs-ui/oojs-ui-apex.svg.rtl.css M lib/oojs-ui/oojs-ui-mediawiki.css M lib/oojs-ui/oojs-ui-mediawiki.js M lib/oojs-ui/oojs-ui-mediawiki.rtl.css M lib/oojs-ui/oojs-ui-mediawiki.svg.css M lib/oojs-ui/oojs-ui-mediawiki.svg.rtl.css M lib/oojs-ui/oojs-ui.js M src/ui/widgets/ve.ui.LinkTargetInputWidget.js 12 files changed, 336 insertions(+), 28 deletions(-) Approvals: Bartosz Dziewoński: Looks good to me, but someone else must approve Jforrester: Looks good to me, approved jenkins-bot: Verified diff --git a/lib/oojs-ui/oojs-ui-apex.css b/lib/oojs-ui/oojs-ui-apex.css index 1ccb25a..02116d0 100644 --- a/lib/oojs-ui/oojs-ui-apex.css +++ b/lib/oojs-ui/oojs-ui-apex.css @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:41Z + * Date: 2014-11-21T22:32:01Z */ .oo-ui-progressBarWidget-slide-frames from { margin-left: -40%; @@ -970,6 +970,9 @@ border-bottom-right-radius: 0.3em; border-top-right-radius: 0.3em; } +.oo-ui-radioSelectWidget { + padding: 0.75em 0 0.5em 0; +} .oo-ui-buttonOptionWidget { display: inline-block; padding: 0; @@ -997,6 +1000,24 @@ .oo-ui-buttonOptionWidget.oo-ui-optionWidget-highlighted { background-color: transparent; } +.oo-ui-radioOptionWidget { + cursor: default; + padding: 0; + background-color: transparent; +} +.oo-ui-radioOptionWidget .oo-ui-radioInputWidget, +.oo-ui-radioOptionWidget .oo-ui-labelElement-label { + display: inline-block; + vertical-align: middle; +} +.oo-ui-radioOptionWidget.oo-ui-optionWidget-selected, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-pressed, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-highlighted { + background-color: transparent; +} +.oo-ui-radioOptionWidget > .oo-ui-labelElement-label { + padding: 0 0.5em; +} .oo-ui-labelWidget { display: inline-block; padding: 0.5em 0; diff --git a/lib/oojs-ui/oojs-ui-apex.js b/lib/oojs-ui/oojs-ui-apex.js index c851ac8..bff36f8 100644 --- a/lib/oojs-ui/oojs-ui-apex.js +++ b/lib/oojs-ui/oojs-ui-apex.js @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:30Z + * Date: 2014-11-21T22:31:51Z */ /* Instantiation */ diff --git a/lib/oojs-ui/oojs-ui-apex.rtl.css b/lib/oojs-ui/oojs-ui-apex.rtl.css index e77b387..c9eab99 100644 --- a/lib/oojs-ui/oojs-ui-apex.rtl.css +++ b/lib/oojs-ui/oojs-ui-apex.rtl.css @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:41Z + * Date: 2014-11-21T22:32:01Z */ .oo-ui-progressBarWidget-slide-frames from { margin-right: -40%; @@ -970,6 +970,9 @@ border-bottom-left-radius: 0.3em; border-top-left-radius: 0.3em; } +.oo-ui-radioSelectWidget { + padding: 0.75em 0 0.5em 0; +} .oo-ui-buttonOptionWidget { display: inline-block; padding: 0; @@ -997,6 +1000,24 @@ .oo-ui-buttonOptionWidget.oo-ui-optionWidget-highlighted { background-color: transparent; } +.oo-ui-radioOptionWidget { + cursor: default; + padding: 0; + background-color: transparent; +} +.oo-ui-radioOptionWidget .oo-ui-radioInputWidget, +.oo-ui-radioOptionWidget .oo-ui-labelElement-label { + display: inline-block; + vertical-align: middle; +} +.oo-ui-radioOptionWidget.oo-ui-optionWidget-selected, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-pressed, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-highlighted { + background-color: transparent; +} +.oo-ui-radioOptionWidget > .oo-ui-labelElement-label { + padding: 0 0.5em; +} .oo-ui-labelWidget { display: inline-block; padding: 0.5em 0; diff --git a/lib/oojs-ui/oojs-ui-apex.svg.css b/lib/oojs-ui/oojs-ui-apex.svg.css index e4305bb..118f062 100644 --- a/lib/oojs-ui/oojs-ui-apex.svg.css +++ b/lib/oojs-ui/oojs-ui-apex.svg.css @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:41Z + * Date: 2014-11-21T22:32:01Z */ .oo-ui-progressBarWidget-slide-frames from { margin-left: -40%; @@ -970,6 +970,9 @@ border-bottom-right-radius: 0.3em; border-top-right-radius: 0.3em; } +.oo-ui-radioSelectWidget { + padding: 0.75em 0 0.5em 0; +} .oo-ui-buttonOptionWidget { display: inline-block; padding: 0; @@ -997,6 +1000,24 @@ .oo-ui-buttonOptionWidget.oo-ui-optionWidget-highlighted { background-color: transparent; } +.oo-ui-radioOptionWidget { + cursor: default; + padding: 0; + background-color: transparent; +} +.oo-ui-radioOptionWidget .oo-ui-radioInputWidget, +.oo-ui-radioOptionWidget .oo-ui-labelElement-label { + display: inline-block; + vertical-align: middle; +} +.oo-ui-radioOptionWidget.oo-ui-optionWidget-selected, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-pressed, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-highlighted { + background-color: transparent; +} +.oo-ui-radioOptionWidget > .oo-ui-labelElement-label { + padding: 0 0.5em; +} .oo-ui-labelWidget { display: inline-block; padding: 0.5em 0; diff --git a/lib/oojs-ui/oojs-ui-apex.svg.rtl.css b/lib/oojs-ui/oojs-ui-apex.svg.rtl.css index 1edcc8a..4aae840 100644 --- a/lib/oojs-ui/oojs-ui-apex.svg.rtl.css +++ b/lib/oojs-ui/oojs-ui-apex.svg.rtl.css @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:41Z + * Date: 2014-11-21T22:32:01Z */ .oo-ui-progressBarWidget-slide-frames from { margin-right: -40%; @@ -970,6 +970,9 @@ border-bottom-left-radius: 0.3em; border-top-left-radius: 0.3em; } +.oo-ui-radioSelectWidget { + padding: 0.75em 0 0.5em 0; +} .oo-ui-buttonOptionWidget { display: inline-block; padding: 0; @@ -997,6 +1000,24 @@ .oo-ui-buttonOptionWidget.oo-ui-optionWidget-highlighted { background-color: transparent; } +.oo-ui-radioOptionWidget { + cursor: default; + padding: 0; + background-color: transparent; +} +.oo-ui-radioOptionWidget .oo-ui-radioInputWidget, +.oo-ui-radioOptionWidget .oo-ui-labelElement-label { + display: inline-block; + vertical-align: middle; +} +.oo-ui-radioOptionWidget.oo-ui-optionWidget-selected, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-pressed, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-highlighted { + background-color: transparent; +} +.oo-ui-radioOptionWidget > .oo-ui-labelElement-label { + padding: 0 0.5em; +} .oo-ui-labelWidget { display: inline-block; padding: 0.5em 0; diff --git a/lib/oojs-ui/oojs-ui-mediawiki.css b/lib/oojs-ui/oojs-ui-mediawiki.css index 770f5dc..1671845 100644 --- a/lib/oojs-ui/oojs-ui-mediawiki.css +++ b/lib/oojs-ui/oojs-ui-mediawiki.css @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:41Z + * Date: 2014-11-21T22:32:01Z */ .oo-ui-progressBarWidget-slide-frames from { margin-left: -40%; @@ -879,6 +879,9 @@ border-bottom-right-radius: 0.3em; border-top-right-radius: 0.3em; } +.oo-ui-radioSelectWidget { + padding: 0.75em 0 0.5em 0; +} .oo-ui-buttonOptionWidget { display: inline-block; padding: 0; @@ -906,6 +909,24 @@ .oo-ui-buttonOptionWidget.oo-ui-optionWidget-highlighted { background-color: transparent; } +.oo-ui-radioOptionWidget { + cursor: default; + padding: 0; + background-color: transparent; +} +.oo-ui-radioOptionWidget .oo-ui-radioInputWidget, +.oo-ui-radioOptionWidget .oo-ui-labelElement-label { + display: inline-block; + vertical-align: middle; +} +.oo-ui-radioOptionWidget.oo-ui-optionWidget-selected, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-pressed, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-highlighted { + background-color: transparent; +} +.oo-ui-radioOptionWidget > .oo-ui-labelElement-label { + padding: 0 0.5em; +} .oo-ui-labelWidget { display: inline-block; } diff --git a/lib/oojs-ui/oojs-ui-mediawiki.js b/lib/oojs-ui/oojs-ui-mediawiki.js index 7ddb170..76d01f3 100644 --- a/lib/oojs-ui/oojs-ui-mediawiki.js +++ b/lib/oojs-ui/oojs-ui-mediawiki.js @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:30Z + * Date: 2014-11-21T22:31:51Z */ /** * @class diff --git a/lib/oojs-ui/oojs-ui-mediawiki.rtl.css b/lib/oojs-ui/oojs-ui-mediawiki.rtl.css index 6ebf6f2..f6527ea 100644 --- a/lib/oojs-ui/oojs-ui-mediawiki.rtl.css +++ b/lib/oojs-ui/oojs-ui-mediawiki.rtl.css @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:41Z + * Date: 2014-11-21T22:32:01Z */ .oo-ui-progressBarWidget-slide-frames from { margin-right: -40%; @@ -879,6 +879,9 @@ border-bottom-left-radius: 0.3em; border-top-left-radius: 0.3em; } +.oo-ui-radioSelectWidget { + padding: 0.75em 0 0.5em 0; +} .oo-ui-buttonOptionWidget { display: inline-block; padding: 0; @@ -906,6 +909,24 @@ .oo-ui-buttonOptionWidget.oo-ui-optionWidget-highlighted { background-color: transparent; } +.oo-ui-radioOptionWidget { + cursor: default; + padding: 0; + background-color: transparent; +} +.oo-ui-radioOptionWidget .oo-ui-radioInputWidget, +.oo-ui-radioOptionWidget .oo-ui-labelElement-label { + display: inline-block; + vertical-align: middle; +} +.oo-ui-radioOptionWidget.oo-ui-optionWidget-selected, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-pressed, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-highlighted { + background-color: transparent; +} +.oo-ui-radioOptionWidget > .oo-ui-labelElement-label { + padding: 0 0.5em; +} .oo-ui-labelWidget { display: inline-block; } diff --git a/lib/oojs-ui/oojs-ui-mediawiki.svg.css b/lib/oojs-ui/oojs-ui-mediawiki.svg.css index 559f99a..0a4ae37 100644 --- a/lib/oojs-ui/oojs-ui-mediawiki.svg.css +++ b/lib/oojs-ui/oojs-ui-mediawiki.svg.css @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:41Z + * Date: 2014-11-21T22:32:01Z */ .oo-ui-progressBarWidget-slide-frames from { margin-left: -40%; @@ -879,6 +879,9 @@ border-bottom-right-radius: 0.3em; border-top-right-radius: 0.3em; } +.oo-ui-radioSelectWidget { + padding: 0.75em 0 0.5em 0; +} .oo-ui-buttonOptionWidget { display: inline-block; padding: 0; @@ -906,6 +909,24 @@ .oo-ui-buttonOptionWidget.oo-ui-optionWidget-highlighted { background-color: transparent; } +.oo-ui-radioOptionWidget { + cursor: default; + padding: 0; + background-color: transparent; +} +.oo-ui-radioOptionWidget .oo-ui-radioInputWidget, +.oo-ui-radioOptionWidget .oo-ui-labelElement-label { + display: inline-block; + vertical-align: middle; +} +.oo-ui-radioOptionWidget.oo-ui-optionWidget-selected, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-pressed, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-highlighted { + background-color: transparent; +} +.oo-ui-radioOptionWidget > .oo-ui-labelElement-label { + padding: 0 0.5em; +} .oo-ui-labelWidget { display: inline-block; } diff --git a/lib/oojs-ui/oojs-ui-mediawiki.svg.rtl.css b/lib/oojs-ui/oojs-ui-mediawiki.svg.rtl.css index e68caf9..3ba634c 100644 --- a/lib/oojs-ui/oojs-ui-mediawiki.svg.rtl.css +++ b/lib/oojs-ui/oojs-ui-mediawiki.svg.rtl.css @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:41Z + * Date: 2014-11-21T22:32:01Z */ .oo-ui-progressBarWidget-slide-frames from { margin-right: -40%; @@ -879,6 +879,9 @@ border-bottom-left-radius: 0.3em; border-top-left-radius: 0.3em; } +.oo-ui-radioSelectWidget { + padding: 0.75em 0 0.5em 0; +} .oo-ui-buttonOptionWidget { display: inline-block; padding: 0; @@ -906,6 +909,24 @@ .oo-ui-buttonOptionWidget.oo-ui-optionWidget-highlighted { background-color: transparent; } +.oo-ui-radioOptionWidget { + cursor: default; + padding: 0; + background-color: transparent; +} +.oo-ui-radioOptionWidget .oo-ui-radioInputWidget, +.oo-ui-radioOptionWidget .oo-ui-labelElement-label { + display: inline-block; + vertical-align: middle; +} +.oo-ui-radioOptionWidget.oo-ui-optionWidget-selected, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-pressed, +.oo-ui-radioOptionWidget.oo-ui-optionWidget-highlighted { + background-color: transparent; +} +.oo-ui-radioOptionWidget > .oo-ui-labelElement-label { + padding: 0 0.5em; +} .oo-ui-labelWidget { display: inline-block; } diff --git a/lib/oojs-ui/oojs-ui.js b/lib/oojs-ui/oojs-ui.js index 6a766f7..87ba532 100644 --- a/lib/oojs-ui/oojs-ui.js +++ b/lib/oojs-ui/oojs-ui.js @@ -1,12 +1,12 @@ /*! - * OOjs UI v0.1.0-pre (23565e7519) + * OOjs UI v0.1.0-pre (8f8896196f) * https://www.mediawiki.org/wiki/OOjs_UI * * Copyright 2011–2014 OOjs Team and other contributors. * Released under the MIT license * http://oojs.mit-license.org * - * Date: 2014-11-21T22:15:30Z + * Date: 2014-11-21T22:31:51Z */ ( function ( OO ) { @@ -9163,12 +9163,12 @@ * @chainable */ OO.ui.InputWidget.prototype.setValue = function ( value ) { - value = this.sanitizeValue( value ); + value = this.cleanUpValue( value ); if ( this.value !== value ) { this.value = value; this.emit( 'change', this.value ); } - // Update the DOM if it has changed. Note that with sanitizeValue, it + // Update the DOM if it has changed. Note that with cleanUpValue, it // is possible for the DOM value to change without this.value changing. if ( this.$input.val() !== this.value ) { this.$input.val( this.value ); @@ -9177,15 +9177,15 @@ }; /** - * Sanitize incoming value. + * Clean up incoming value. * * Ensures value is a string, and converts undefined and null to empty string. * * @private * @param {string} value Original value - * @return {string} Sanitized value + * @return {string} Cleaned up value */ -OO.ui.InputWidget.prototype.sanitizeValue = function ( value ) { +OO.ui.InputWidget.prototype.cleanUpValue = function ( value ) { if ( value === undefined || value === null ) { return ''; } else if ( this.inputFilter ) { @@ -9460,6 +9460,74 @@ widget.setValue( widget.$input.prop( 'checked' ) ); } ); } +}; + +/** + * Radio input widget. + * + * Radio buttons only make sense as a set, and you probably want to use the OO.ui.RadioSelectWidget + * class instead of using this class directly. + * + * This class doesn't make it possible to learn whether the radio button is selected ("pressed"). + * + * @class + * @extends OO.ui.InputWidget + * + * @constructor + * @param {Object} [config] Configuration options + * @param {boolean} [config.selected=false] Whether the radio button is initially selected + */ +OO.ui.RadioInputWidget = function OoUiRadioInputWidget( config ) { + // Parent constructor + OO.ui.RadioInputWidget.super.call( this, config ); + + // Initialization + this.$element.addClass( 'oo-ui-radioInputWidget' ); + this.setSelected( config.selected !== undefined ? config.selected : false ); +}; + +/* Setup */ + +OO.inheritClass( OO.ui.RadioInputWidget, OO.ui.InputWidget ); + +/* Methods */ + +/** + * Get input element. + * + * @private + * @return {jQuery} Input element + */ +OO.ui.RadioInputWidget.prototype.getInputElement = function () { + return this.$( '<input type="radio" />' ); +}; + +/** + * @inheritdoc + */ +OO.ui.RadioInputWidget.prototype.onEdit = function () { + // RadioInputWidget doesn't track its state. +}; + +/** + * Set selection state of this radio button. + * + * @param {boolean} state Whether the button is selected + * @chainable + */ +OO.ui.RadioInputWidget.prototype.setSelected = function ( state ) { + // RadioInputWidget doesn't track its state. + this.$input.prop( 'checked', state ); + return this; +}; + +/** + * Check if this radio button is selected. + * + * @return {boolean} Radio is selected + */ +OO.ui.RadioInputWidget.prototype.isSelected = function () { + return this.$input.prop( 'checked' ); }; /** @@ -10236,6 +10304,55 @@ if ( this.constructor.static.selectable ) { this.setActive( state ); } + + return this; +}; + +/** + * Option widget that looks like a radio button. + * + * Use together with OO.ui.RadioSelectWidget. + * + * @class + * @extends OO.ui.OptionWidget + * @mixins OO.ui.ButtonElement + * + * @constructor + * @param {Mixed} data Option data + * @param {Object} [config] Configuration options + */ +OO.ui.RadioOptionWidget = function OoUiRadioOptionWidget( data, config ) { + // Parent constructor + OO.ui.RadioOptionWidget.super.call( this, data, config ); + + // Properties + this.radio = new OO.ui.RadioInputWidget( { value: data } ); + + // Initialization + this.$element + .addClass( 'oo-ui-radioOptionWidget' ) + .prepend( this.radio.$element ); +}; + +/* Setup */ + +OO.inheritClass( OO.ui.RadioOptionWidget, OO.ui.OptionWidget ); + +/* Static Properties */ + +OO.ui.RadioOptionWidget.static.highlightable = false; + +OO.ui.RadioOptionWidget.static.pressable = false; + +/* Methods */ + +/** + * @inheritdoc + */ +OO.ui.RadioOptionWidget.prototype.setSelected = function ( state ) { + OO.ui.RadioOptionWidget.super.prototype.setSelected.call( this, state ); + + this.radio.setSelected( state ); return this; }; @@ -11477,6 +11594,29 @@ OO.inheritClass( OO.ui.ButtonSelectWidget, OO.ui.SelectWidget ); /** + * Select widget containing radio button options. + * + * Use together with OO.ui.RadioOptionWidget. + * + * @class + * @extends OO.ui.SelectWidget + * + * @constructor + * @param {Object} [config] Configuration options + */ +OO.ui.RadioSelectWidget = function OoUiRadioSelectWidget( config ) { + // Parent constructor + OO.ui.RadioSelectWidget.super.call( this, config ); + + // Initialization + this.$element.addClass( 'oo-ui-radioSelectWidget' ); +}; + +/* Setup */ + +OO.inheritClass( OO.ui.RadioSelectWidget, OO.ui.SelectWidget ); + +/** * Overlaid menu of options. * * Menus are clipped to the visible viewport. They do not provide a control for opening or closing diff --git a/src/ui/widgets/ve.ui.LinkTargetInputWidget.js b/src/ui/widgets/ve.ui.LinkTargetInputWidget.js index f1dbaa8..11741a6 100644 --- a/src/ui/widgets/ve.ui.LinkTargetInputWidget.js +++ b/src/ui/widgets/ve.ui.LinkTargetInputWidget.js @@ -76,7 +76,7 @@ */ ve.ui.LinkTargetInputWidget.prototype.setValue = function ( value ) { // Keep annotation in sync with value - value = this.sanitizeValue( value ); + value = this.cleanUpValue( value ); if ( value === '' ) { this.annotation = null; } else { -- To view, visit https://gerrit.wikimedia.org/r/174814 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: If10927a4e595de9b6d0ad45182470e84f335683c Gerrit-PatchSet: 2 Gerrit-Project: VisualEditor/VisualEditor Gerrit-Branch: master Gerrit-Owner: Bartosz Dziewoński <[email protected]> Gerrit-Reviewer: Bartosz Dziewoński <[email protected]> Gerrit-Reviewer: Jforrester <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
