mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor
synced 2024-09-25 11:16:51 +00:00
Only validate required template params after blur
Bug: 72096 Change-Id: Ib699f8d89af02c9109a5711bd2c3ab4ef664d8ea
This commit is contained in:
parent
af84e6c35b
commit
d087052ada
|
@ -17,7 +17,7 @@
|
|||
* @param {Object} [config] Configuration options
|
||||
*/
|
||||
ve.ui.MWParameterPage = function VeUiMWParameterPage( parameter, name, config ) {
|
||||
var paramName = parameter.getName();
|
||||
var paramName = parameter.getName(), page = this;
|
||||
|
||||
// Configuration initialization
|
||||
config = ve.extendObject( {
|
||||
|
@ -48,12 +48,12 @@ ve.ui.MWParameterPage = function VeUiMWParameterPage( parameter, name, config )
|
|||
.connect( this, { change: 'onValueInputChange' } );
|
||||
|
||||
if ( this.parameter.isRequired() ) {
|
||||
this.valueInput.$input.prop( 'required', true );
|
||||
this.valueInput.$element.on( 'DOMNodeInsertedIntoDocument', function () {
|
||||
// Don't start validating until the input is in the document ready to go,
|
||||
// we don't want it to start off marked as invalid.
|
||||
this.valueInput.setValidation( 'non-empty' );
|
||||
}.bind( this ) );
|
||||
this.valueInput.$input
|
||||
.prop( 'required', true )
|
||||
.on( 'blur', function () {
|
||||
page.valueInput.setValidation( 'non-empty' );
|
||||
page.valueInput.setValidityFlag();
|
||||
} );
|
||||
}
|
||||
|
||||
this.removeButton = new OO.ui.ButtonWidget( {
|
||||
|
|
Loading…
Reference in a new issue