Changeset 6286


Ignore:
Timestamp:
01/02/11 16:21:53 (3 years ago)
Author:
alfonsoml
Message:

#6522 Simplifications in indent and list plugins.

Location:
CKEditor/trunk/_source/plugins
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/_source/plugins/indent/plugin.js

    r6273 r6286  
    1 /* 
     1/* 
    22Copyright (c) 2003-2010, CKSource - Frederico Knabben. All rights reserved. 
    33For licensing, see LICENSE.html or http://ckeditor.com/license 
     
    1010(function() 
    1111{ 
    12         var listNodeNames = { ol : 1, ul : 1 }; 
    13  
    14         var isNotWhitespaces = CKEDITOR.dom.walker.whitespaces( true ), 
     12        var listNodeNames = { ol : 1, ul : 1 }, 
     13                isNotWhitespaces = CKEDITOR.dom.walker.whitespaces( true ), 
    1514                isNotBookmark = CKEDITOR.dom.walker.bookmark( false, true ); 
    16  
    17         function setState( editor, state ) 
    18         { 
    19                 editor.getCommand( this.name ).setState( state ); 
    20         } 
    2115 
    2216        function onSelectionChange( evt ) 
     
    2822 
    2923                if ( list ) 
    30                                 return setState.call( this, editor, CKEDITOR.TRISTATE_OFF ); 
     24                                return this.setState( CKEDITOR.TRISTATE_OFF ); 
    3125 
    3226                if ( !this.useIndentClasses && this.name == 'indent' ) 
    33                         return setState.call( this, editor, CKEDITOR.TRISTATE_OFF ); 
     27                        return this.setState( CKEDITOR.TRISTATE_OFF ); 
    3428 
    3529                var path = evt.data.path, 
    3630                        firstBlock = path.block || path.blockLimit; 
    3731                if ( !firstBlock ) 
    38                         return setState.call( this, editor, CKEDITOR.TRISTATE_DISABLED ); 
     32                        return this.setState( CKEDITOR.TRISTATE_DISABLED ); 
    3933 
    4034                if ( this.useIndentClasses ) 
     
    4943                        if ( ( this.name == 'outdent' && !indentStep ) || 
    5044                                        ( this.name == 'indent' && indentStep == editor.config.indentClasses.length ) ) 
    51                                 return setState.call( this, editor, CKEDITOR.TRISTATE_DISABLED ); 
    52                         return setState.call( this, editor, CKEDITOR.TRISTATE_OFF ); 
     45                                return this.setState( CKEDITOR.TRISTATE_DISABLED ); 
     46                        return this.setState( CKEDITOR.TRISTATE_OFF ); 
    5347                } 
    5448                else 
     
    5852                                indent = 0; 
    5953                        if ( indent <= 0 ) 
    60                                 return setState.call( this, editor, CKEDITOR.TRISTATE_DISABLED ); 
    61                         return setState.call( this, editor, CKEDITOR.TRISTATE_OFF ); 
     54                                return this.setState( CKEDITOR.TRISTATE_DISABLED ); 
     55                        return this.setState( CKEDITOR.TRISTATE_OFF ); 
    6256                } 
    6357        } 
     
    355349                { 
    356350                        // Register commands. 
    357                         var indent = new indentCommand( editor, 'indent' ), 
    358                                 outdent = new indentCommand( editor, 'outdent' ); 
    359                         editor.addCommand( 'indent', indent ); 
    360                         editor.addCommand( 'outdent', outdent ); 
     351                        var indent = editor.addCommand( 'indent', new indentCommand( editor, 'indent' ) ), 
     352                                outdent = editor.addCommand( 'outdent', new indentCommand( editor, 'outdent' ) ); 
    361353 
    362354                        // Register the toolbar buttons. 
  • CKEditor/trunk/_source/plugins/list/plugin.js

    r6268 r6286  
    204204        }; 
    205205 
    206         function setState( editor, state ) 
    207         { 
    208                 editor.getCommand( this.name ).setState( state ); 
    209         } 
    210  
    211206        function onSelectionChange( evt ) 
    212207        { 
     
    214209                        blockLimit = path.blockLimit, 
    215210                        elements = path.elements, 
    216                         element; 
     211                        element, 
     212                        i; 
    217213 
    218214                // Grouping should only happen under blockLimit.(#3940). 
    219                 for ( var i = 0 ; i < elements.length && ( element = elements[ i ] ) 
     215                for ( i = 0 ; i < elements.length && ( element = elements[ i ] ) 
    220216                          && !element.equals( blockLimit ); i++ ) 
    221217                { 
    222218                        if ( listNodeNames[ elements[i].getName() ] ) 
    223                         { 
    224                                 return setState.call( this, evt.editor, 
    225                                                 this.type == elements[i].getName() ? CKEDITOR.TRISTATE_ON : CKEDITOR.TRISTATE_OFF ); 
    226                         } 
    227                 } 
    228  
    229                 return setState.call( this, evt.editor, CKEDITOR.TRISTATE_OFF ); 
     219                                return this.setState( this.type == elements[i].getName() ? CKEDITOR.TRISTATE_ON : CKEDITOR.TRISTATE_OFF ); 
     220                } 
     221 
     222                return this.setState( CKEDITOR.TRISTATE_OFF ); 
    230223        } 
    231224 
     
    493486                                        if ( enclosedNode && enclosedNode.is 
    494487                                                && this.type == enclosedNode.getName() ) 
    495                                                         setState.call( this, editor, CKEDITOR.TRISTATE_ON ); 
     488                                                        this.setState( CKEDITOR.TRISTATE_ON ); 
    496489                                } 
    497490                        } 
     
    694687                { 
    695688                        // Register commands. 
    696                         var numberedListCommand = new listCommand( 'numberedlist', 'ol' ), 
    697                                 bulletedListCommand = new listCommand( 'bulletedlist', 'ul' ); 
    698                         editor.addCommand( 'numberedlist', numberedListCommand ); 
    699                         editor.addCommand( 'bulletedlist', bulletedListCommand ); 
     689                        var numberedListCommand = editor.addCommand( 'numberedlist', new listCommand( 'numberedlist', 'ol' ) ), 
     690                                bulletedListCommand = editor.addCommand( 'bulletedlist', new listCommand( 'bulletedlist', 'ul' ) ); 
    700691 
    701692                        // Register the toolbar button. 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy