r113325 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r113324‎ | r113325 | r113326 >
Date:23:33, 7 March 2012
Author:inez
Status:deferred
Tags:
Comment:
Get rid of showCursorAt method. And replace calls to it with calls to showCursor.
Modified paths:
  • /trunk/extensions/VisualEditor/modules/ve/ce/ve.ce.Surface.js (modified) (history)

Diff [purge]

Index: trunk/extensions/VisualEditor/modules/ve/ce/ve.ce.Surface.js
@@ -108,7 +108,7 @@
109109 this.clearPollData();
110110
111111 // show cursor
112 - _this.showCursorAt(range.to);
 112+ _this.showCursor(range.to);
113113 } else {
114114 if ( method === 'set' ) {
115115 this.addInsertionAnnotation( annotation );
@@ -183,7 +183,7 @@
184184 _this.clearPollData();
185185
186186 // place cursor
187 - _this.showCursorAt( selection.start );
 187+ _this.showCursor( selection.start );
188188 }, 1 );
189189 }
190190 };
@@ -214,7 +214,7 @@
215215 _this.clearPollData();
216216
217217 // place cursor
218 - _this.showCursorAt( insertionPoint + _this.clipboard[key].length );
 218+ _this.showCursor( insertionPoint + _this.clipboard[key].length );
219219 } else {
220220 alert('i can only handle copy/paste from hybrid surface. sorry. :(');
221221 }
@@ -424,7 +424,7 @@
425425 newOffset = this.documentView.model.getRelativeContentOffset(
426426 globalOffset, -1
427427 );
428 - this.showCursorAt(newOffset);
 428+ this.showCursor(newOffset);
429429 e.preventDefault();
430430 }
431431 }
@@ -440,7 +440,7 @@
441441 newOffset = this.documentView.model.getRelativeContentOffset(
442442 globalOffset, 1
443443 );
444 - this.showCursorAt(newOffset);
 444+ this.showCursor(newOffset);
445445 e.preventDefault();
446446 }
447447 }
@@ -498,53 +498,6 @@
499499 }
500500 };
501501
502 -ve.ce.Surface.prototype.showCursorAt = function( offset ) {
503 - var $node = this.documentView.getNodeFromOffset( offset ).$,
504 - current = [$node.contents(), 0],
505 - stack = [current],
506 - node,
507 - localOffset,
508 - index = this.documentView.getOffsetFromNode( $node.data('view') ) + 1;
509 -
510 - while ( stack.length > 0 ) {
511 - if ( current[1] >= current[0].length ) {
512 - stack.pop();
513 - current = stack[ stack.length - 1 ];
514 - continue;
515 - }
516 - var item = current[0][current[1]],
517 - $item = current[0].eq( current[1] );
518 -
519 - if ( item.nodeType === 3 ) {
520 - var length = item.textContent.length;
521 - if ( offset >= index && offset <= index + length ) {
522 - node = item;
523 - localOffset = offset - index;
524 - break;
525 - } else {
526 - index += length;
527 - }
528 - } else if ( item.nodeType === 1 ) {
529 - if ( $( item ).attr('contentEditable') === 'false' ) {
530 - index += 1;
531 - } else {
532 - stack.push( [$item.contents(), 0] );
533 - current[1]++;
534 - current = stack[stack.length-1];
535 - continue;
536 - }
537 - }
538 - current[1]++;
539 - }
540 - var range = document.createRange();
541 - range.collapsed = true;
542 - range.setStart( node, localOffset );
543 - var sel = window.getSelection();
544 - sel.removeAllRanges();
545 - sel.addRange( range );
546 -};
547 -
548 -
549502 /**
550503 * @method
551504 * @returns {ve.Range} Current selection range

Status & tagging log