Changeset 1360

Show
Ignore:
Timestamp:
2008-01-18 17:52:29 (2 years ago)
Author:
fredck
Message:

Small cleanup in the code.

Location:
FCKeditor/branches/features/floating_dialog/editor/_source/internals
Files:
2 modified

Legend:

Unmodified
Added
Removed

  • FCKeditor/branches/features/floating_dialog/editor/_source/internals/fckdomtools.js

    r1329 r1360  
    947947        { 
    948948                var currentElement = element ; 
     949 
    949950                while ( currentElement != FCKTools.GetElementDocument( currentElement ).documentElement ) 
    950951                { 
    951952                        var currentWindow = FCKTools.GetElementWindow( currentElement ) ; 
     953 
    952954                        if ( this.GetCurrentElementStyle( currentWindow, currentElement, 'position' ) != 'static' ) 
    953955                                return currentElement ; 
    954                         if ( currentElement == FCKTools.GetElementDocument( currentElement ).documentElement && 
    955                                 currentWindow != w ) 
     956 
     957                        if ( currentElement == FCKTools.GetElementDocument( currentElement ).documentElement 
     958                                        && currentWindow != w ) 
    956959                                currentElement = currentWindow.frameElement ; 
    957960                        else 
    958961                                currentElement = currentElement.parentNode ; 
    959962                } 
     963 
    960964                return null ; 
    961965        }, 

  • FCKeditor/branches/features/floating_dialog/editor/_source/internals/fcktools.js

    r1332 r1360  
    612612 
    613613/** 
    614  * Utility function for binding the "this" reference to an object for a function. 
     614 * Binding the "this" reference to an object for a function. 
    615615 */ 
    616616FCKTools.Bind = function( subject, func ) 
    … …  
    620620 
    621621/** 
    622  * Retrieve the correct "empty iframe" URL for the current browser, which causes the  
    623  * minimum fuzz (e.g. security warnings in HTTPS, DNS error in IE5.5, etc.) for that 
    624  * browser, but updates the iframe. 
     622 * Retrieve the correct "empty iframe" URL for the current browser, which 
     623 * causes the minimum fuzz (e.g. security warnings in HTTPS, DNS error in 
     624 * IE5.5, etc.) for that browser, making the iframe ready to DOM use whithout 
     625 * having to loading an external file. 
    625626 */ 
    626627FCKTools.GetVoidUrl = function() 
    … …  
    629630                return "javascript: void( function(){" + 
    630631                        "document.open();" + 
    631                         "document.write('<html><head><title></title></head><body></body></html>');" +  
     632                        "document.write('<html><head><title></title></head><body></body></html>');" + 
    632633                        "document.domain = '" + FCK_RUNTIME_DOMAIN + "';" + 
    633634                        "document.close();" + 
    634635                        "}() ) ;"; 
     636 
    635637        if ( FCKBrowserInfo.IsIE ) 
    636638        { 
    637                 if ( FCKBrowserInfo.IsIE7 )     // IE7+ 
    638                         return "" ; 
    639                 else if ( FCKBrowserInfo.IsIE6 )        // IE6+ 
    640                         return "javascript: '';" ; 
    641                 else            // IE5.5 
    642                         return ""; 
    643         } 
    644         return "javascript: void(0);" ; 
    645 } 
     639                if ( FCKBrowserInfo.IsIE7 || !FCKBrowserInfo.IsIE6 ) 
     640                        return "" ;                                     // IE7+ / IE5.5 
     641                else 
     642                        return "javascript: '';" ;      // IE6+ 
     643        } 
     644 
     645        return "javascript: void(0);" ;         // All other browsers. 
     646}