Changeset 13696

Timestamp:

05/12/08 08:26:13 (6 months ago)

Author:

dante

Message:

fixes #5626 and refs #4058 - removes code working around sarafi race condition. With the newest test cases update
to use "real" link attributes and no-magic css loading, these blocks are unnecessary. tested safari 3.1.1 on a variety
of hosts with different throughput and load times. tested built version vs raw trunk. all tabs align on startup.

Location:

dijit/trunk

Files:

• layout/TabContainer.js (modified) (1 diff)
• _base.js (modified) (1 diff)

dijit/trunk/layout/TabContainer.js

@@ -53 +53 @@
                 this.tablist.startup();
                 this.inherited(arguments);
-
-                if(dojo.isSafari){
-                        // sometimes safari 3.0.3 miscalculates the height of the tab labels, see #4058
-                        setTimeout(dojo.hitch(this, "layout"), 0);
-                }
 
                 if(dojo.isIE && !this.isLeftToRight() && this.tabPosition == "right-h" &&

dijit/trunk/_base.js

@@ -11 +11 @@
 dojo.require("dijit._base.wai");
 dojo.require("dijit._base.window");
-
-//      FIXME: Find a better way of solving this bug!
-if(dojo.isSafari){
-        //      Ugly-ass hack to solve bug #5626 for 1.1; basically force Safari to re-layout.
-        //      Note that we can't reliably use dojo.addOnLoad here because this bug is basically
-        //              a timing / race condition; so instead we use window.onload.
-        dojo.connect(window, "load", function(){
-                window.resizeBy(1,0);
-                setTimeout(function(){ window.resizeBy(-1,0); }, 10);
-        });
-}