http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89858

Revision: 89858
Author:   brion
Date:     2011-06-10 23:09:52 +0000 (Fri, 10 Jun 2011)
Log Message:
-----------
CodeEditor: fix to avoid selecting when inserting non-samples such as character 
insertions
Logging some stub func calls

Modified Paths:
--------------
    trunk/extensions/CodeEditor/modules/jquery.codeEditor.js

Modified: trunk/extensions/CodeEditor/modules/jquery.codeEditor.js
===================================================================
--- trunk/extensions/CodeEditor/modules/jquery.codeEditor.js    2011-06-10 
22:36:18 UTC (rev 89857)
+++ trunk/extensions/CodeEditor/modules/jquery.codeEditor.js    2011-06-10 
23:09:52 UTC (rev 89858)
@@ -75,8 +75,10 @@
                return;
        },
        'saveSelection': function() {
+               mw.log('codeEditor stub function saveSelection called');
        },
        'restoreSelection': function() {
+               mw.log('codeEditor stub function restoreSelection called');
        },
        /**
         * Sets up the iframe in place of the textarea to allow more advanced 
operations
@@ -152,6 +154,7 @@
         */
 
        'getElementAtCursor': function() {
+               mw.log('codeEditor stub function getElementAtCursor called');
        },
 
        /**
@@ -171,10 +174,10 @@
                var sel = context.codeEditor.getSelection();
                var range = sel.getRange();
                var selText = context.fn.getSelection();
-               var selectAfter = false;
+               var isSample = false;
                if ( !selText ) {
                        selText = options.peri;
-                       selectAfter = true;
+                       isSample = true;
                } else if ( options.replace ) {
                        selText = options.peri;
                }
@@ -182,7 +185,7 @@
                text += selText;
                text += options.post;
                context.codeEditor.insert( text );
-               if ( selectAfter ) {
+               if ( isSample && options.selectPeri && !options.splitlines ) {
                        // May esplode if anything has newlines, be warned. :)
                        range.setStart( range.start.row, range.start.column + 
options.pre.length );
                        range.setEnd( range.start.row, range.start.column + 
selText.length );
@@ -195,6 +198,7 @@
         * DO NOT CALL THIS DIRECTLY, use $.textSelection( 'functionname', 
options ) instead
         */
        'getCaretPosition': function( options ) {
+               mw.log('codeEditor stub function getCaretPosition called');
        },
        /**
         * Sets the selection of the content
@@ -237,7 +241,7 @@
         * DO NOT CALL THIS DIRECTLY, use $.textSelection( 'functionname', 
options ) instead
         */
        'scrollToCaretPosition': function( options ) {
-               //context.fn.scrollToTop( context.fn.getElementAtCursor(), true 
);
+               mw.log('codeEditor stub function scrollToCaretPosition called');
                return context.$textarea;
        },
        /**
@@ -248,6 +252,7 @@
         * @param force If true, scroll the element even if it's already visible
         */
        'scrollToTop': function( $element, force ) {
+               mw.log('codeEditor stub function scrollToTop called');
        }
 } );
 


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

Reply via email to