Trevor Parscal has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/130421

Change subject: Cleanup focus/select/clear
......................................................................

Cleanup focus/select/clear

Depends on I324dfc2 in OOUI

Change-Id: Id58d199c7ce3332a104ab2a5c395561ab94ae060
---
M lib/ve
M modules/ve-mw/ui/dialogs/ve.ui.MWMediaInsertDialog.js
M modules/ve-mw/ui/dialogs/ve.ui.MWReferenceDialog.js
M modules/ve-mw/ui/inspectors/ve.ui.MWExtensionInspector.js
4 files changed, 5 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/21/130421/1

diff --git a/lib/ve b/lib/ve
index d21c272..ed84663 160000
--- a/lib/ve
+++ b/lib/ve
-Subproject commit d21c272adfbba03a02f7242e02ceed2d3024a559
+Subproject commit ed846639a22319156526b1c92c8e93fe4a2bbdc3
diff --git a/modules/ve-mw/ui/dialogs/ve.ui.MWMediaInsertDialog.js 
b/modules/ve-mw/ui/dialogs/ve.ui.MWMediaInsertDialog.js
index 82c6ba4..399a1d8 100644
--- a/modules/ve-mw/ui/dialogs/ve.ui.MWMediaInsertDialog.js
+++ b/modules/ve-mw/ui/dialogs/ve.ui.MWMediaInsertDialog.js
@@ -115,7 +115,7 @@
                // Initialization
                // This must be done only after there are proper
                // sources defined
-               this.search.getQuery().$input.focus().select();
+               this.search.getQuery().focus().select();
                this.search.getResults().selectItem();
                this.search.getResults().highlightItem();
        }, this ) );
@@ -209,7 +209,7 @@
                ] ).collapseRangeToEnd().select();
        }
 
-       this.search.clear();
+       this.search.getQuery.setValue( '' );
 
        // Parent method
        ve.ui.Dialog.prototype.teardown.call( this, data );
diff --git a/modules/ve-mw/ui/dialogs/ve.ui.MWReferenceDialog.js 
b/modules/ve-mw/ui/dialogs/ve.ui.MWReferenceDialog.js
index a0bc7b8..050d183 100644
--- a/modules/ve-mw/ui/dialogs/ve.ui.MWReferenceDialog.js
+++ b/modules/ve-mw/ui/dialogs/ve.ui.MWReferenceDialog.js
@@ -266,7 +266,7 @@
                this.insertButton.$element.hide();
                this.selectButton.$element.hide();
                this.panels.setItem( this.searchPanel );
-               this.search.getQuery().$input.focus().select();
+               this.search.getQuery().focus().select();
        } } );
        this.backButton.connect( this, { 'click': function () {
                this.backButton.$element.hide();
@@ -354,7 +354,7 @@
                this.referenceModel.updateInternalItem( surfaceModel );
        }
 
-       this.search.clear();
+       this.search.getQuery().setValue( '' );
        this.referenceSurface.destroy();
        this.referenceSurface = null;
        this.referenceModel = null;
diff --git a/modules/ve-mw/ui/inspectors/ve.ui.MWExtensionInspector.js 
b/modules/ve-mw/ui/inspectors/ve.ui.MWExtensionInspector.js
index fd22b0b..b6f31a6 100644
--- a/modules/ve-mw/ui/inspectors/ve.ui.MWExtensionInspector.js
+++ b/modules/ve-mw/ui/inspectors/ve.ui.MWExtensionInspector.js
@@ -111,7 +111,7 @@
        ve.ui.Inspector.prototype.ready.call( this );
 
        // Focus the input
-       this.input.$input.focus().select();
+       this.input.focus().select();
 };
 
 /**

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id58d199c7ce3332a104ab2a5c395561ab94ae060
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Trevor Parscal <tpars...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to