Esanders has uploaded a new change for review.
https://gerrit.wikimedia.org/r/230085
Change subject: Switch to using surface model focus/blur events
......................................................................
Switch to using surface model focus/blur events
Depends on Ibcda9a79 in core.
Change-Id: Ieb2f99d7d1d24c24a0f00cf7c33e23b8206bf753
---
M modules/ve-mw/init/targets/ve.init.mw.MobileArticleTarget.js
1 file changed, 19 insertions(+), 11 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor
refs/changes/85/230085/1
diff --git a/modules/ve-mw/init/targets/ve.init.mw.MobileArticleTarget.js
b/modules/ve-mw/init/targets/ve.init.mw.MobileArticleTarget.js
index 4e4088a..3d1518a 100644
--- a/modules/ve-mw/init/targets/ve.init.mw.MobileArticleTarget.js
+++ b/modules/ve-mw/init/targets/ve.init.mw.MobileArticleTarget.js
@@ -80,24 +80,32 @@
// Parent method
ve.init.mw.MobileArticleTarget.super.prototype.onSurfaceReady.apply(
this, arguments );
- this.getSurface().getModel().connect( this, { select: 'onSurfaceSelect'
} );
- this.onSurfaceSelect();
+ var surfaceModel = this.getSurface().getModel();
+ surfaceModel.connect( this, {
+ blur: 'onSurfaceBlur',
+ focus: 'onSurfaceFocus'
+ } );
+ this[surfaceModel.getSelection().isNull() ? 'onSurfaceBlur' :
'onSurfaceFocus']();
this.events.trackActivationComplete();
};
/**
- * Handle surface select events
+ * Handle surface blur events
*/
-ve.init.mw.MobileArticleTarget.prototype.onSurfaceSelect = function () {
+ve.init.mw.MobileArticleTarget.prototype.onSurfaceBlur = function () {
var toolbar = this.getToolbar();
- if ( this.getSurface().getModel().getSelection().isNull() ) {
- toolbar.$group.addClass( 'oo-ui-element-hidden' );
- toolbar.$actions.removeClass( 'oo-ui-element-hidden' );
- } else {
- toolbar.$group.removeClass( 'oo-ui-element-hidden' );
- toolbar.$actions.addClass( 'oo-ui-element-hidden' );
- }
+ toolbar.$group.addClass( 'oo-ui-element-hidden' );
+ toolbar.$actions.removeClass( 'oo-ui-element-hidden' );
+};
+
+/**
+ * Handle surface focus events
+ */
+ve.init.mw.MobileArticleTarget.prototype.onSurfaceFocus = function () {
+ var toolbar = this.getToolbar();
+ toolbar.$group.removeClass( 'oo-ui-element-hidden' );
+ toolbar.$actions.addClass( 'oo-ui-element-hidden' );
};
/**
--
To view, visit https://gerrit.wikimedia.org/r/230085
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ieb2f99d7d1d24c24a0f00cf7c33e23b8206bf753
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Esanders <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
