IE flicker has been killed and setAuto now works in Firefox and IE.
authorJohn Resig <jeresig@gmail.com>
Mon, 24 Jul 2006 03:09:35 +0000 (03:09 +0000)
committerJohn Resig <jeresig@gmail.com>
Mon, 24 Jul 2006 03:09:35 +0000 (03:09 +0000)
fx/fx.js
jquery/jquery.js

index 44633ab..bd34c30 100644 (file)
--- a/fx/fx.js
+++ b/fx/fx.js
@@ -121,12 +121,20 @@ jQuery.fn.extend({
 jQuery.extend({
 
        setAuto: function(e,p) {
 jQuery.extend({
 
        setAuto: function(e,p) {
+               // Remember the original height
                var a = e.style[p];
                var a = e.style[p];
-               var o = jQuery.css(e,p);
-               e.style[p] = "auto";
-               var n = jQuery.css(e,p);
-               if ( o != n )
-                       e.style[p] = a;
+
+               // Figure out the size of the height right now
+               var o = jQuery.curCSS(e,p,1);
+
+               // Set the height to auto
+               e.style[p] = e.currentStyle ? "" : "auto";
+
+               // See what the size of "auto" is
+               var n = jQuery.curCSS(e,p,1);
+
+               // Revert back to the original size
+               if ( o != n && n != "auto" ) e.style[p] = a;
        },
        
        speed: function(s,o,i) {
        },
        
        speed: function(s,o,i) {
@@ -198,8 +206,10 @@ jQuery.extend({
                                if (window.ActiveXObject)
                                        y.filter = "alpha(opacity=" + z.now*100 + ")";
                                y.opacity = z.now;
                                if (window.ActiveXObject)
                                        y.filter = "alpha(opacity=" + z.now*100 + ")";
                                y.opacity = z.now;
-                       } else
-                               y[prop] = z.now+"px";
+
+                       // My hate for IE will never die
+                       } else if ( parseInt(z.now) )
+                                       y[prop] = parseInt(z.now) + "px";
                };
        
                // Figure out the maximum number to run to
                };
        
                // Figure out the maximum number to run to
@@ -231,7 +241,13 @@ jQuery.extend({
                        z.el.orig[prop] = this.cur();
 
                        if ( !y[prop] ) z.o.auto = true;
                        z.el.orig[prop] = this.cur();
 
                        if ( !y[prop] ) z.o.auto = true;
+
                        z.custom(0,z.max());
                        z.custom(0,z.max());
+
+                       // Stupid IE, look what you made me do
+                       if ( prop != "opacity" )
+                               y[prop] = "1px";
+
                        y.display = "block";
                };
        
                        y.display = "block";
                };
        
@@ -242,7 +258,8 @@ jQuery.extend({
                        // Remember where we started, so that we can go back to it later
                        z.el.orig[prop] = this.cur();
 
                        // Remember where we started, so that we can go back to it later
                        z.el.orig[prop] = this.cur();
 
-                       z.o.hide = true;        
+                       z.o.hide = true;
+
                        // Begin the animation
                        z.custom(z.cur(),0);
                };
                        // Begin the animation
                        z.custom(z.cur(),0);
                };
@@ -265,33 +282,27 @@ jQuery.extend({
                                // Stop the timer
                                clearInterval(z.timer);
                                z.timer = null;
                                // Stop the timer
                                clearInterval(z.timer);
                                z.timer = null;
-       
+
                                z.now = lastNum;
                                z.a();
                                z.now = lastNum;
                                z.a();
+
+                               // Hide the element if the "hide" operation was done
+                               if ( z.o.hide ) y.display = 'none';
        
                                // Reset the overflow
                                y.overflow = z.oldOverflow;
 
        
                                // Reset the overflow
                                y.overflow = z.oldOverflow;
 
-                               // If the element was shown, and not using a custom number,
-                               // set its height and/or width to auto
-                               if ( (prop == "height" || prop == "width") && z.o.auto )
-                                       jQuery.setAuto( z.el, prop );
-
                                // If a callback was provided, execute it
                                // If a callback was provided, execute it
-                               if( z.o.complete && z.o.complete.constructor == Function ) {
-       
-                                       // Yes, this is a weird place for this, but it needs to be executed
-                                       // only once per cluster of effects.
-                                       // If the element is, effectively, hidden - hide it
-                                       if ( z.o.hide ) y.display = "none";
-       
+                               if( z.o.complete && z.o.complete.constructor == Function )
                                        // Execute the complete function
                                        z.o.complete.apply( z.el );
                                        // Execute the complete function
                                        z.o.complete.apply( z.el );
-                               }
 
                                // Reset the property, if the item has been hidden
                                if ( z.o.hide )
                                        y[ prop ] = z.el.orig[ prop ].constructor == Number && prop != "opacity" ? z.el.orig[prop] + "px" : z.el.orig[prop];
 
                                // Reset the property, if the item has been hidden
                                if ( z.o.hide )
                                        y[ prop ] = z.el.orig[ prop ].constructor == Number && prop != "opacity" ? z.el.orig[prop] + "px" : z.el.orig[prop];
+
+                               // set its height and/or width to auto
+                               jQuery.setAuto( z.el, prop );
                        } else {
                                // Figure out where in the animation we are and set the number
                                var p = (t - this.startTime) / z.o.duration;
                        } else {
                                // Figure out where in the animation we are and set the number
                                var p = (t - this.startTime) / z.o.duration;
index 7fd1146..f987c86 100644 (file)
@@ -62,7 +62,7 @@ function jQuery(a,c) {
 
        // Watch for when an array is passed in
        this.get( a.constructor == Array || a.length && a[0] != undefined && a[0].nodeType ?
 
        // Watch for when an array is passed in
        this.get( a.constructor == Array || a.length && a[0] != undefined && a[0].nodeType ?
-               // Assume that it's an array of DOM Elements
+               // Assume that it is an array of DOM Elements
                jQuery.merge( a, [] ) :
 
                // Find the matching elements and save them for later
                jQuery.merge( a, [] ) :
 
                // Find the matching elements and save them for later
@@ -1181,10 +1181,10 @@ jQuery.extend({
                return jQuery.curCSS( e, p );
        },
 
                return jQuery.curCSS( e, p );
        },
 
-       curCSS: function(e,p) {
+       curCSS: function(e,p,force) {
                var r;
        
                var r;
        
-               if (e.style[p])
+               if (!force && e.style[p])
                        r = e.style[p];
                else if (e.currentStyle)
                        r = e.currentStyle[p];
                        r = e.style[p];
                else if (e.currentStyle)
                        r = e.currentStyle[p];