Changeset 4918


Ignore:
Timestamp:
01/12/10 19:46:21 (5 years ago)
Author:
alfonsoml
Message:

#4640 Little optimizations in the filebrowser plugin

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r4917 r4918  
    4848                <li><a href="http://dev.fckeditor.net/ticket/4637">#4637</a> : Table cells' 'nowrap' attribute is not loaded to cell property dialog. Thanks to pomu0325.</li> 
    4949                <li><a href="http://dev.fckeditor.net/ticket/4724">#4724</a> : Using the mouse to insert a link in IE might create incorrect results.</li> 
     50                <li><a href="http://dev.fckeditor.net/ticket/4640">#4640</a> : Small optimizations for the fileBrowser plugin.</li> 
    5051        </ul> 
    5152        <h3> 
  • CKEditor/trunk/_source/plugins/filebrowser/plugin.js

    r4858 r4918  
    336336                } 
    337337 
    338                 return ( definition.getContents( tabId ).get( elementId ).filebrowser && definition.getContents( tabId ).get( elementId ).filebrowser.url ); 
     338                var elementFileBrowser = definition.getContents( tabId ).get( elementId ).filebrowser; 
     339                return ( elementFileBrowser && elementFileBrowser.url ); 
    339340        } 
    340341 
     
    367368                        CKEDITOR.on( 'dialogDefinition', function( evt ) 
    368369                        { 
     370                                var definition = evt.data.definition, 
     371                                        element; 
    369372                                // Associate filebrowser to elements with 'filebrowser' attribute. 
    370                                 for ( var i in evt.data.definition.contents ) 
     373                                for ( var i in definition.contents ) 
    371374                                { 
    372                                         attachFileBrowser( evt.editor, evt.data.name, evt.data.definition, evt.data.definition.contents[ i ].elements ); 
    373                                         if ( evt.data.definition.contents[ i ].hidden && evt.data.definition.contents[ i ].filebrowser ) 
     375                                        element = definition.contents[ i ] ; 
     376                                        attachFileBrowser( evt.editor, evt.data.name, definition, element.elements ); 
     377                                        if ( element.hidden && element.filebrowser ) 
    374378                                        { 
    375                                                 evt.data.definition.contents[ i ].hidden = 
    376                                                         !isConfigured( evt.data.definition, evt.data.definition.contents[ i ][ 'id' ], evt.data.definition.contents[ i ].filebrowser ); 
     379                                                element.hidden = !isConfigured( definition, element[ 'id' ], element.filebrowser ); 
    377380                                        } 
    378381                                } 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy