Inez has uploaded a new change for review.

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


Change subject: Very small cleanup of onCut method in ve.ce.Surface.
......................................................................

Very small cleanup of onCut method in ve.ce.Surface.

Use ve.bind in order to change context within which callback in called -
- reduces a need of intro ducting new local variable.

Change-Id: I75ece695548c87073dd22e5e7ec80057d7132d22
---
M modules/ve/ce/ve.ce.Surface.js
1 file changed, 7 insertions(+), 11 deletions(-)


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

diff --git a/modules/ve/ce/ve.ce.Surface.js b/modules/ve/ce/ve.ce.Surface.js
index 776b38e..1d41be1 100644
--- a/modules/ve/ce/ve.ce.Surface.js
+++ b/modules/ve/ce/ve.ce.Surface.js
@@ -371,28 +371,24 @@
  * @param {jQuery.Event} e
  */
 ve.ce.Surface.prototype.onCut = function ( e ) {
-       var surface = this;
-
        this.surfaceObserver.stop();
-
        this.onCopy( e );
-
-       setTimeout( function () {
+       setTimeout( ve.bind( function () {
                var selection,
                        tx;
 
                // We don't like how browsers cut, so let's undo it and do it 
ourselves.
                document.execCommand( 'undo', false, false );
 
-               selection = surface.model.getSelection();
+               selection = this.model.getSelection();
 
                // Transact
-               tx = ve.dm.Transaction.newFromRemoval( 
surface.documentView.model, selection );
-               surface.model.change( tx, new ve.Range( selection.start ) );
+               tx = ve.dm.Transaction.newFromRemoval( this.documentView.model, 
selection );
+               this.model.change( tx, new ve.Range( selection.start ) );
 
-               surface.surfaceObserver.clear();
-               surface.surfaceObserver.start();
-       }, 1 );
+               this.surfaceObserver.clear();
+               this.surfaceObserver.start();
+       }, this ), 1 );
 };
 
 /**

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I75ece695548c87073dd22e5e7ec80057d7132d22
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Inez <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to