Ticket #2816: 2816_4.patch

File 2816_4.patch, 20.5 KB (added by Garry Yao, 12 years ago)

Fix leading spaces

  • _source/core/editor.js

     
    380380                execCommand : function( commandName, data )
    381381                {
    382382                        var command = this.getCommand( commandName );
     383                        var exeResult;
     384                        var commandEvent =
     385
     386                        /**
     387                         * The event data for this command
     388                         * @type FCKEDITOR.command.Event
     389                         */
     390                        {
     391                                name :commandName,
     392                                command :command
     393                        };
    383394                        if ( command && command.state != CKEDITOR.TRISTATE_DISABLED )
    384                                 return command.exec( this, data );
     395                        {
     396                                this.fire( 'beforeCommandExec' , commandEvent );
     397                                exeResult = command.exec( this , data );
     398                                this.fire( 'afterCommandExec' , commandEvent );
     399                                return exeResult;
     400                        }
    385401
    386402                        // throw 'Unknown command name "' + commandName + '"';
    387403                        return false;
  • _source/plugins/dialog/plugin.js

     
    205205
    206206                this.on( 'ok', function( evt )
    207207                        {
     208                                this._.editor.fire( 'beforeDialogClose' );
    208209                                iterContents( function( item )
    209210                                        {
    210211                                                if ( item.validate )
     
    234235
    235236                this.on( 'cancel', function( evt )
    236237                        {
     238                                this._.editor.fire( 'beforeDialogClose' );
    237239                                iterContents( function( item )
    238240                                        {
    239241                                                if ( item.isChanged() )
     
    461463                                        } );
    462464
    463465                        // Save editor selection and grab the focus.
    464                         if ( !this._.parentDialog )
    465                                 this.saveSelection();
    466466                        this._.dummyText.focus();
    467467                        this._.dummyText.$.select();
    468468
     469                        this._.editor.fire( 'beforeDialogShow', this );
    469470                        // Execute onLoad for the first show.
    470471                        this.fireOnce( 'load', {} );
    471472                        this.fire( 'show', {} );
     
    575576                                CKEDITOR.document.removeListener( 'keydown', accessKeyDownHandler );
    576577                                CKEDITOR.document.removeListener( 'keyup', accessKeyUpHandler );
    577578
    578                                 // Restore focus and (if not already restored) selection in the editing area.
    579                                 this.restoreSelection();
    580                                 this._.editor.focus();
    581579                        }
    582580                        else
    583581                                CKEDITOR.dialog._.currentZIndex -= 10;
     
    818816                getParentEditor : function()
    819817                {
    820818                        return this._.editor;
    821                 },
    822 
    823                 /**
    824                  * Saves the current selection position in the editor.
    825                  * This function is automatically called when a non-nested dialog is opened,
    826                  * but it may also be called by event handlers in dialog definition.
    827                  * @example
    828                  */
    829                 saveSelection : function()
    830                 {
    831                         if ( this._.editor.mode )
    832                         {
    833                                 var selection = new CKEDITOR.dom.selection( this._.editor.document );
    834                                 this._.selectedRanges = selection.getRanges();
    835                         }
    836                 },
    837 
    838                 /**
    839                  * Clears the saved selection in the dialog object.
    840                  * This function should be called if the dialog's code has already changed the
    841                  * current selection position because the dialog closed. (e.g. at onOk())
    842                  * @example
    843                  */
    844                 clearSavedSelection : function()
    845                 {
    846                         delete this._.selectedRanges;
    847                 },
     819                }
    848820
    849                 /**
    850                  * Restores the editor's selection from the previously saved position in this
    851                  * dialog.
    852                  * This function is automatically called when a non-nested dialog is closed,
    853                  * but it may also be called by event handlers in dialog definition.
    854                  * @example
    855                  */
    856                 restoreSelection : function()
    857                 {
    858                         if ( this._.editor.mode && this._.selectedRanges )
    859                                 ( new CKEDITOR.dom.selection( this._.editor.document ) ).selectRanges( this._.selectedRanges );
    860                 }
    861821        };
    862822
    863823        CKEDITOR.tools.extend( CKEDITOR.dialog,
  • _source/plugins/link/dialogs/link.js

     
    636636                onShow : function()
    637637                {
    638638                        this.fakeObj = false;
    639                         // IE BUG: Selection must be in the editor for getSelection() to work.
    640                         this.restoreSelection();
    641639                        var editor = this.getParentEditor(),
    642640                                selection = editor.getSelection(),
    643641                                ranges = selection.getRanges(),
     
    696694                                {
    697695                                        loadLink.apply( this, [ editor, selection, ranges, element ] );
    698696                                        selection.selectElement( element );
    699                                         this.saveSelection();
    700697                                }
    701698
    702699                                element = rangeRoot.getAscendant( 'img', true );
     
    706703                                        element = editor.fakeobjects.restoreElement( this.fakeObj );
    707704                                        loadLink.apply( this, [ editor, selection, ranges, element ] );
    708705                                        selection.selectElement( this.fakeObj );
    709                                         this.saveSelection();
    710706                                }
    711707                        }
    712708
     
    803799
    804800                        if ( !this._.selectedElement )
    805801                        {
    806                                 // IE BUG: Selection must be in the editor for getSelection() to work.
    807                                 this.restoreSelection();
    808                                 this.clearSavedSelection();
    809 
    810802                                // Create element if current selection is collapsed.
    811803                                var selection = editor.getSelection(),
    812804                                        ranges = selection.getRanges();
  • _source/plugins/link/dialogs/anchor.js

     
    88        // Function called in onShow to load selected element.
    99        var loadElements = function( editor, selection, ranges, element )
    1010        {
    11                 this.saveSelection();
    1211                this.editMode = true;
    1312                this.editObj = element;
    1413
     
    3635                        // Insert a new anchor.
    3736                        if ( !this.editMode )
    3837                        {
    39                                 // It doesn't work with IE.
    40                                 this.restoreSelection();
    41                                 this.clearSavedSelection();
    4238
    4339                                var fakeElement = editor.fakeobjects.protectElement( this.editObj );
    4440                                editor.insertElement( fakeElement );
     
    5349                        this.fakeObj = false;
    5450                        this.editMode = false;
    5551
    56                         // IE BUG: Selection must be in the editor for getSelection() to work.
    57                         this.restoreSelection();
    58 
    5952                        var editor = this.getParentEditor(),
    6053                                selection = editor.getSelection(),
    6154                                ranges = selection.getRanges();
     
    7164                                        element = editor.fakeobjects.restoreElement( this.fakeObj );
    7265                                        loadElements.apply( this, [ editor, selection, ranges, element ] );
    7366                                        selection.selectElement( this.fakeObj );
    74                                         this.saveSelection();
    7567                                }
    7668                        }
    7769                        this.pushDefault();
  • _source/plugins/specialchar/dialogs/specialchar.js

     
    140140                                                                                                target = target.$;
    141141                                                                                                if ( value = target.getAttribute( 'value' ) )
    142142                                                                                                {
    143                                                                                                         this.getDialog().restoreSelection();
    144143                                                                                                        editor.insertHtml( value );
    145144                                                                                                        this.getDialog().hide();
    146145                                                                                                }
  • _source/plugins/image/dialogs/image.js

     
    341341        {
    342342                this.editObj[ 'image' ] = false;
    343343                this.editMode[ 'image' ] = false;               // Default:  Create a new Image
    344                 this.saveSelection();
    345344
    346345                this.setValueOf( "Link", "txtUrl", "" );
    347346                this.setValueOf( "Link", "cmbTarget", "" );
     
    501500                        // Insert a new Image.
    502501                        if ( this.editMode[ 'image' ] == false )
    503502                        {
    504                                 // It doesn't work with IE.
    505                                 this.restoreSelection();
    506                                 this.clearSavedSelection();
    507503
    508504                                if ( useLink )
    509505                                        //Insert a new Link.
     
    559555                        this.dimensionsInStyle[ 'height' ] = false;
    560556                        this.forceResetSize = false;
    561557
    562                         // IE BUG: Selection must be in the editor for getSelection() to work.
    563                         this.restoreSelection();
    564558
    565559                        var editor = this.getParentEditor(),
    566560                                selection = editor.getSelection(),
     
    602596                                {
    603597                                        loadElements.apply( this, [ editor, selection, ranges, element ] );
    604598                                        selection.selectElement( element );
    605                                         this.saveSelection();
    606599                                }
    607600                        }
    608601                        updateOriginal( this, false );
  • _source/plugins/forms/dialogs/textarea.js

     
    1717        // Function called in onShow to load selected element.
    1818        var loadSelection = function( editor, selection, ranges, element )
    1919        {
    20                 this.saveSelection();
    2120                this.editObj = element;
    2221                this.editMode = true;
    2322
     
    4746                        // Default: create a new element.
    4847                        this.editMode = false;
    4948
    50                         // IE BUG: Selection must be in the editor for getSelection() to work.
    51                         this.restoreSelection();
    5249
    5350                        var editor = this.getParentEditor(),
    5451                                selection = editor.getSelection(),
     
    8077
    8178                        if ( this.editMode == false )
    8279                        {
    83                                 this.restoreSelection();
    84                                 this.clearSavedSelection();
    8580                                editor.insertElement( this.editObj );
    8681                        }
    8782                        return true;
  • _source/plugins/forms/dialogs/radio.js

     
    1717        // Function called in onShow to load selected element.
    1818        var loadSelection = function( editor, selection, ranges, element )
    1919        {
    20                 this.saveSelection();
    2120                this.editObj = element;
    2221                this.editMode = true;
    2322
     
    4443                        this.editMode = false;
    4544                        this.changedAttibutes = new Array();
    4645
    47                         // IE BUG: Selection must be in the editor for getSelection() to work.
    48                         this.restoreSelection();
    4946
    5047                        var editor = this.getParentEditor(),
    5148                                selection = editor.getSelection(),
     
    7976                        if ( this.editMode == false )
    8077                        {
    8178                                // It doesn't work with IE.
    82                                 this.restoreSelection();
    83                                 this.clearSavedSelection();
    8479                                editor.insertElement( this.editObj );
    8580                        }
    8681                        return true;
  • _source/plugins/forms/dialogs/form.js

     
    1717        // Function called in onShow to load selected element.
    1818        var loadSelection = function( editor, selection, ranges, element )
    1919        {
    20                 this.saveSelection();
    2120                this.editObj = element;
    2221                this.editMode = true;
    2322
     
    5453
    5554                        if ( this.editMode == false )
    5655                        {
    57                                 this.restoreSelection();
    58                                 this.clearSavedSelection();
    5956                                editor.insertElement( this.editObj );
    6057                        }
    6158                        return true;
     
    6663                        this.editMode = false;
    6764                        this.changedAttibutes = new Array();
    6865
    69                         // IE BUG: Selection must be in the editor for getSelection() to work.
    70                         this.restoreSelection();
    7166
    7267                        var editor = this.getParentEditor(),
    7368                                selection = editor.getSelection(),
  • _source/plugins/forms/dialogs/textfield.js

     
    1717        // Function called in onShow to load selected element.
    1818        var loadSelection = function( editor, selection, ranges, element )
    1919        {
    20                 this.saveSelection();
    2120                this.editObj = element;
    2221                this.editMode = true;
    2322
     
    4948                        // Default: create a new element.
    5049                        this.editMode = false;
    5150
    52                         // IE BUG: Selection must be in the editor for getSelection() to work.
    53                         this.restoreSelection();
    5451
    5552                        var editor = this.getParentEditor(),
    5653                                selection = editor.getSelection(),
     
    8683
    8784                        if ( this.editMode == false )
    8885                        {
    89                                 this.restoreSelection();
    90                                 this.clearSavedSelection();
    9186                                editor.insertElement( this.editObj );
    9287                        }
    9388                        return true;
  • _source/plugins/forms/dialogs/select.js

     
    4444
    4545                dialog.editObj = element;
    4646                dialog.editMode = true;
    47                 dialog.saveSelection();
    4847
    4948                dialog.getContentElement( 'info', 'txtName' ).setValue( dialog.editObj.getAttribute( "name" ) );
    5049                dialog.getContentElement( 'info', 'txtSize' ).setValue( dialog.editObj.getAttribute( "size" ) );
     
    7473                        this.editMode = false;
    7574                        this.changedAttibutes = new Array();
    7675
    77                         // IE BUG: Selection must be in the editor for getSelection() to work.
    78                         this.restoreSelection();
    7976
    8077                        var editor = this.getParentEditor(),
    8178                                selection = editor.getSelection(),
     
    9188                                {
    9289                                        loadSelection.apply( this, [ editor, selection, ranges, element ] );
    9390                                        selection.selectElement( element );
    94                                         this.saveSelection();
    9591                                }
    9692                        }
    9793
     
    139135
    140136                        if ( this.editMode == false )
    141137                        {
    142                                 this.restoreSelection();
    143138                                parentEditor.insertElement( this.editObj );
    144139                        }
    145140                        return true;
  • _source/plugins/forms/dialogs/hiddenfield.js

     
    1717        // Function called in onShow to load selected element.
    1818        var loadSelection = function( editor, selection, ranges, element )
    1919        {
    20                 this.saveSelection();
    2120                this.editObj = element;
    2221                this.editMode = true;
    2322
     
    4140                        this.editMode = false;
    4241                        this.changedAttibutes = new Array();
    4342
    44                         // IE BUG: Selection must be in the editor for getSelection() to work.
    45                         this.restoreSelection();
    4643
    4744                        var editor = this.getParentEditor(),
    4845                                selection = editor.getSelection(),
     
    7471
    7572                        if ( this.editMode == false )
    7673                        {
    77                                 this.restoreSelection();
    78                                 this.clearSavedSelection();
    7974                                editor.insertElement( this.editObj );
    8075                        }
    8176                        return true;
  • _source/plugins/forms/dialogs/checkbox.js

     
    1010                minHeight : 230,
    1111                onShow : function()
    1212                {
    13                         // IE BUG: Selection must be in the editor for getSelectedElement()
    14                         // to work.
    15                         this.restoreSelection();
    16 
    1713                        var element = this.getParentEditor().getSelection().getSelectedElement();
    1814
    1915                        if ( element && element.getAttribute( 'type' ) == "checkbox" )
     
    3935
    4036                        if ( isInsertMode )
    4137                        {
    42                                 this.restoreSelection();
    43                                 this.clearSavedSelection();
    4438                                editor.insertElement( element );
    4539                        }
    4640                },
  • _source/plugins/forms/dialogs/button.js

     
    2626                this.setValueOf( 'info', 'txtValue', value );
    2727                this.setValueOf( 'info', 'txtType', type );
    2828
    29                 this.saveSelection();
    3029                this.editMode = true;
    3130                this.editObj = element;
    3231
     
    4342                        // Default: create a new element.
    4443                        this.editMode = false;
    4544
    46                         // IE BUG: Selection must be in the editor for getSelection() to work.
    47                         this.restoreSelection();
    4845
    4946                        var editor = this.getParentEditor(),
    5047                                selection = editor.getSelection(),
     
    8077
    8178                        if ( this.editMode == false )
    8279                        {
    83                                 this.restoreSelection();
    84                                 this.clearSavedSelection();
    8580                                editor.insertElement( this.editObj );
    8681                        }
    8782                        return true;
  • _source/plugins/flash/dialogs/flash.js

     
    163163        var loadElements = function( editor, selection, ranges, fakeElement, realElement )
    164164        {
    165165                selection.selectElement( fakeElement );
    166                 this.saveSelection();
    167166                this.editMode[ 'fakeobject' ] = realElement.getName();
    168167                this.editObj[ 'fakeobject' ] = fakeElement;
    169168
     
    326325                        this.dimensionsInStyle[ 'width' ] = false;
    327326                        this.dimensionsInStyle[ 'height' ] = false;
    328327
    329                         // IE BUG: Selection must be in the editor for getSelection() to work.
    330                         this.restoreSelection();
    331328
    332329                        var editor = this.getParentEditor(),
    333330                                selection = editor.getSelection(),
     
    415412                                else
    416413                                        this.editObj[ 'fakeobject' ] = CKEDITOR.plugins.fakeobjects.protectElement( this.editObj[ 'object' ] );
    417414
    418                                 // It doesn't work with IE.
    419                                 this.restoreSelection();
    420                                 this.clearSavedSelection();
    421415                                editor.insertElement( this.editObj[ 'fakeobject' ] );
    422416                        }
    423417                        else
  • _source/plugins/table/dialogs/table.js

     
    1616                onShow : function()
    1717                {
    1818                        // Detect if there's a selected table.
    19                         this.restoreSelection();
    2019                        var selection = editor.getSelection(),
    2120                                ranges = selection.getRanges(),
    2221                                me = this,
     
    148147                        // Insert the table element if we're creating one.
    149148                        if ( !this._.selectedElement )
    150149                        {
    151                                 this.restoreSelection();
    152150                                editor.insertElement( table );
    153151                                this.clearSavedSelection();
    154152                        }
  • _source/plugins/selection/plugin.js

     
    9090                                                // IE is the only to provide the "selectionchange"
    9191                                                // event.
    9292                                                editor.document.on( 'selectionchange', checkSelectionChangeTimeout, editor );
     93
     94                                                //Destory previous saved selection.
     95                                                editor.document.on( 'selectionchange', function ()
     96                                                {
     97                                                        editor.dropSavedRanges();
     98                                                });
     99
     100                                                // Save selection on editor window blur
     101                                                ( new CKEDITOR.dom.element( editor.document.$.body ) ).on( 'beforedeactivate',
     102                                                function (){
     103                                                        editor.saveSelection();
     104                                                });
    93105                                        }
    94106                                        else
    95107                                        {
     
    102114                                        }
    103115                                });
    104116
    105                         editor.addCommand( 'selectAll', selectAllCmd );
     117                        //restore selection for commands.
     118                        if ( CKEDITOR.env.ie )
     119                        {
     120                                editor.on( 'beforeCommandExec',  editor.restoreSelection,  editor);
     121                                editor.on( 'beforeDialogShow',  editor.restoreSelection,  editor);
     122                                editor.on( 'beforeDialogClose',  editor.restoreSelection,  editor);
     123                        }
     124                        editor.addCommand( 'selectAll',  selectAllCmd );
    106125                        editor.ui.addButton( 'SelectAll',
    107126                                {
    108127                                        label : editor.lang.selectAll,
     
    141160        return retval;
    142161};
    143162
     163//Selection restore  APIs for IE, see #2816
     164if( CKEDITOR.env.ie )
     165{
     166        CKEDITOR.tools.extend( CKEDITOR.editor.prototype,  {
     167
     168                /**
     169                 * Record the current selection status range of this document.
     170                 * Note: This save doesn't resistant to dom changes, resort to undo/redo plugin for such need.
     171                 */
     172                saveSelection : function()
     173                {
     174                        if ( this.mode === 'wysiwyg' && !this._.selectedRanges )
     175                        {
     176                                var selection = new CKEDITOR.dom.selection( this.document );
     177                                this._.selectedRanges = selection.getRanges();
     178                        }
     179                },
     180
     181                /**
     182                 * Clears the saved selection ranges when expired.
     183                 */
     184                dropSavedRanges : function()
     185                {
     186                        if( this._.selectedRanges )
     187                        {
     188                                delete this._.selectedRanges;
     189                        }
     190                },
     191
     192                /**
     193                 * Retrieve the last saved ranges
     194                 * @return {CKEDITOR.dom.range}
     195                 */
     196                getSavedRange: function()
     197                {
     198                        return this._.selectionRanges||null;
     199                },
     200
     201                /**
     202                 * Restores the last ranges record and select the ranges in document.
     203                 */
     204                restoreSelection : function()
     205                {
     206                        if ( this._.selectedRanges && this.mode === 'wysiwyg' )
     207                        {
     208                                ( new CKEDITOR.dom.selection( this.document ) ).selectRanges( this._.selectedRanges );
     209                        }
     210                }
     211});
     212}
    144213/**
    145214 * Gets the current selection from the document.
    146215 * @returns {CKEDITOR.dom.selection} A selection object.
  • _source/plugins/smiley/dialogs/smiley.js

     
    5454                        else if ( targetName != 'img' )
    5555                                return;
    5656
    57                         this.getDialog().restoreSelection();
    58                        
    5957                        var src = target.getAttribute( 'src' ),
    6058                                title = target.getAttribute( 'title' );
    6159                       
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy