wikimedia/mediawiki-extensions-VisualEditor
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor synced 2024-09-23 18:28:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use new selectItemByData method where possible

Browse source 
Change-Id: I32f213bf2977623a0bde83fb9119fda42a1e9848
This commit is contained in:
Ed Sanders 2015-05-08 16:15:16 +01:00
parent 14f9edc493
commit 431f2abcfd
5 changed files with 18 additions and 46 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

46
modules/ve-mw/ui/dialogs/ve.ui.MWMediaDialog.js
View file

@ -773,16 +773,14 @@ ve.ui.MWMediaDialog.prototype.confirmSelectedImage = function () {
/**
* Handle image model alignment change
*
* @param {string} alignment Image alignment
*/
ve.ui.MWMediaDialog.prototype.onImageModelAlignmentChange = function ( alignment ) {
var item;
alignment = alignment || 'none';
item = alignment !== 'none' ? this.positionSelect.getItemFromData( alignment ) : null;
// Select the item without triggering the 'choose' event
this.positionSelect.selectItem( item );
this.positionSelect.selectItemByData( alignment !== 'none' ? alignment : undefined );
this.positionCheckbox.setSelected( alignment !== 'none' );
this.checkChanged();
@ -790,13 +788,11 @@ ve.ui.MWMediaDialog.prototype.onImageModelAlignmentChange = function ( alignment
/**
* Handle image model type change
*
* @param {string} alignment Image alignment
*/
ve.ui.MWMediaDialog.prototype.onImageModelTypeChange = function ( type ) {
var item = type ? this.typeSelect.getItemFromData( type ) : null;
this.typeSelect.selectItem( item );
this.typeSelect.selectItemByData( type );
this.borderCheckbox.setDisabled(
!this.imageModel.isBorderable()
@ -1079,39 +1075,19 @@ ve.ui.MWMediaDialog.prototype.attachImageModel = function () {
this.sizeWidget.updateDefaultDimensions();
// Set initial alt text
this.altTextInput.setValue(
this.imageModel.getAltText()
);
this.altTextInput.setValue( this.imageModel.getAltText() );
// Set initial alignment
this.positionSelect.setDisabled(
!this.imageModel.isAligned()
);
this.positionSelect.selectItem(
this.imageModel.isAligned() ?
this.positionSelect.getItemFromData(
this.imageModel.getAlignment()
) :
null
);
this.positionCheckbox.setSelected(
this.imageModel.isAligned()
);
this.positionSelect.setDisabled( !this.imageModel.isAligned() );
this.positionSelect.selectItemByData( this.imageModel.isAligned() && this.imageModel.getAlignment() );
this.positionCheckbox.setSelected( this.imageModel.isAligned() );
// Border flag
this.borderCheckbox.setDisabled(
!this.imageModel.isBorderable()
);
this.borderCheckbox.setSelected(
this.imageModel.isBorderable() && this.imageModel.hasBorder()
);
this.borderCheckbox.setDisabled( !this.imageModel.isBorderable() );
this.borderCheckbox.setSelected( this.imageModel.isBorderable() && this.imageModel.hasBorder() );
// Type select
this.typeSelect.selectItem(
this.typeSelect.getItemFromData(
this.imageModel.getType() || 'none'
)
);
this.typeSelect.selectItemByData( this.imageModel.getType() || 'none' );
this.isSettingUpModel = false;
};

4
modules/ve-mw/ui/dialogs/ve.ui.MWTemplateDialog.js
View file

@ -339,9 +339,7 @@ ve.ui.MWTemplateDialog.prototype.getSelectedNode = function ( data ) {
*/
ve.ui.MWTemplateDialog.prototype.setPageByName = function ( name ) {
if ( this.bookletLayout.isOutlined() ) {
this.bookletLayout.getOutline().selectItem(
this.bookletLayout.getOutline().getItemFromData( name )
);
this.bookletLayout.getOutline().selectItemByData( name );
} else {
this.bookletLayout.setPage( name );
}

8
modules/ve-mw/ui/inspectors/ve.ui.MWLinkAnnotationInspector.js
View file

@ -98,7 +98,7 @@ ve.ui.MWLinkAnnotationInspector.prototype.onInternalLinkChange = function ( anno
!this.allowProtocolInInternal &&
ve.init.platform.getExternalLinkUrlProtocolsRegExp().test( href )
) {
this.linkTypeSelect.selectItem( this.linkTypeSelect.getItemFromData( 'external' ) );
this.linkTypeSelect.selectItemByData( 'external' );
}
};
@ -115,10 +115,8 @@ ve.ui.MWLinkAnnotationInspector.prototype.createAnnotationInput = function () {
ve.ui.MWLinkAnnotationInspector.prototype.getSetupProcess = function ( data ) {
return ve.ui.MWLinkAnnotationInspector.super.prototype.getSetupProcess.call( this, data )
.next( function () {
this.linkTypeSelect.selectItem(
this.linkTypeSelect.getItemFromData(
this.initialAnnotation instanceof ve.dm.MWExternalLinkAnnotation ? 'external' : 'internal'
)
this.linkTypeSelect.selectItemByData(
this.initialAnnotation instanceof ve.dm.MWExternalLinkAnnotation ? 'external' : 'internal'
);
this.annotationInput.setAnnotation( this.initialAnnotation );
}, this );

4
modules/ve-mw/ui/pages/ve.ui.MWAdvancedSettingsPage.js
View file

@ -231,11 +231,11 @@ ve.ui.MWAdvancedSettingsPage.prototype.setup = function ( metaList ) {
advancedSettingsPage = this;
// Indexing items
indexingField.selectItem( indexingField.getItemFromData( indexingType ) );
indexingField.selectItemByData( indexingType );
this.indexingOptionTouched = false;
// New section edit link items
newSectionEditField.selectItem( newSectionEditField.getItemFromData( newSectionEditLinkType ) );
newSectionEditField.selectItemByData( newSectionEditLinkType );
this.newSectionEditLinkOptionTouched = false;
this.enableDisplayTitleCheckbox.setSelected( !!displayTitleItem );

2
modules/ve-mw/ui/pages/ve.ui.MWSettingsPage.js
View file

@ -250,7 +250,7 @@ ve.ui.MWSettingsPage.prototype.setup = function ( metaList ) {
settingsPage = this;
// Table of Contents items
tableOfContentsField.selectItem( tableOfContentsField.getItemFromData( tableOfContentsMode ) );
tableOfContentsField.selectItemByData( tableOfContentsMode );
this.tableOfContentsTouched = false;
// Redirect items (disabled states set by change event)