Changeset 13157

Show
Ignore:
Timestamp:
03/20/08 08:07:37 (8 months ago)
Author:
ttrenka
Message:

A fix for the *Really* Temporary Ugly Fix(tm), forcing Safari to resize itself; connect to window.onload instead of using dojo.addOnLoad. !strict, refs #5626.

Files:
1 modified

Legend:

Unmodified
Added
Removed

  • dijit/trunk/_base.js

    r13149 r13157  
    1212dojo.require("dijit._base.window"); 
    1313 
    14 //      TODO: Find a better way of solving this bug! 
    15 //      Ugly-ass hack to solve bug #5626 for 1.1; basically force Safari to re-layout. 
     14//      FIXME: Find a better way of solving this bug! 
    1615if(dojo.isSafari){ 
    17         dojo.addOnLoad(function(){ 
    18                 setTimeout(function(){ 
    19                         window.resizeBy(1,0); 
    20                         setTimeout(function(){ window.resizeBy(-1,0); }, 10); 
    21                 }, 10); 
     16        //      Ugly-ass hack to solve bug #5626 for 1.1; basically force Safari to re-layout. 
     17        //      Note that we can't reliably use dojo.addOnLoad here because this bug is basically 
     18        //              a timing / race condition; so instead we use window.onload. 
     19        dojo.connect(window, "load", function(){ 
     20                window.resizeBy(1,0); 
     21                setTimeout(function(){ window.resizeBy(-1,0); }, 10); 
    2222        }); 
    2323}