Bug Tracker

#8167 closed bug (duplicate)

Opened February 03, 2011 05:47PM UTC

Closed February 03, 2011 06:03PM UTC

Last modified March 14, 2012 07:35PM UTC

Solution for incorrect left, top, width, height css when $(obj) is hidden

Reported by: svcorp77 Owned by:
Priority: low Milestone: 1.next
Component: css Version: 1.5
Keywords: Cc:
Blocked by: Blocking:
Description

See http://jsfiddle.net/addyosmani/F9LKa/

The 1.4.2 output of your code is:

display: block; width: 100px; height: 100px; left: 100px; top: 100px; position: absolute; opacity: 0; overflow: hidden; 100px 100px animation complete display: block; width: 420px; height: 150px; left: 38px; top: 97px; position: absolute; opacity: 1; 38px 97px

The 1.4.3 output is:

display: none; width: 100px; height: 100px; left: 0px; top: 0px; position: absolute; opacity: 0; overflow: hidden; 0px 0px animation complete display: none; width: 420px; height: 150px; left: 38px; top: 97px; position: absolute; opacity: 1; 0px 0px

And the Current Git Version output is:

display: none; width: 100px; height: 100px; left: 0px; top: 0px; position: absolute; opacity: 0; overflow: hidden; 0px 0px animation complete display: none; width: 420px; height: 150px; left: 38px; top: 97px; position: absolute; opacity: 1; 0px 0px

Here is solution for:

jQuery JavaScript Library v1.5, string 7610, function jQuery.fx.prototype.update

jQuery version > 1.4.3: the same mods.

                // Set display property to block for height/width animations
		if ( ( this.prop === "height" || this.prop === "width" ) && this.elem.style ) {
			this.elem.style.display = "block";
		}

Please, make this changes next update.

Attachments (0)
Change History (3)

Changed February 03, 2011 05:54PM UTC by rwaldron comment:1

description: See http://jsfiddle.net/addyosmani/F9LKa/[[BR]] \ [[BR]] \ The 1.4.2 output of your code is:[[BR]] \ [[BR]] \ display: block; width: 100px; height: 100px; left: 100px; top: 100px; position: absolute; opacity: 0; overflow: hidden; 100px 100px animation complete display: block; width: 420px; height: 150px; left: 38px; top: 97px; position: absolute; opacity: 1; 38px 97px[[BR]] \ [[BR]] \ The 1.4.3 output is:[[BR]] \ [[BR]] \ display: none; width: 100px; height: 100px; left: 0px; top: 0px; position: absolute; opacity: 0; overflow: hidden; 0px 0px animation complete display: none; width: 420px; height: 150px; left: 38px; top: 97px; position: absolute; opacity: 1; 0px 0px[[BR]] \ [[BR]] \ And the Current Git Version output is:[[BR]] \ [[BR]] \ display: none; width: 100px; height: 100px; left: 0px; top: 0px; position: absolute; opacity: 0; overflow: hidden; 0px 0px animation complete display: none; width: 420px; height: 150px; left: 38px; top: 97px; position: absolute; opacity: 1; 0px 0px [[BR]] \ [[BR]] \ Here is solution for:[[BR]] \ jQuery JavaScript Library v1.5, string 7610, function jQuery.fx.prototype.update[[BR]] \ jQuery version > 1.4.3: the same mods.[[BR]] \ \ {{{ \ // Set display property to block for height/width animations \ if ( ( this.prop === "height" || this.prop === "width" ) && this.elem.style ) { \ this.elem.style.display = "block"; \ } \ \ }}} \ Please, make this changes next update. → See http://jsfiddle.net/addyosmani/F9LKa/ [[BR]] \ [[BR]] \ The 1.4.2 output of your code is: [[BR]] \ [[BR]] \ display: block; width: 100px; height: 100px; left: 100px; top: 100px; position: absolute; opacity: 0; overflow: hidden; 100px 100px animation complete display: block; width: 420px; height: 150px; left: 38px; top: 97px; position: absolute; opacity: 1; 38px 97px[[BR]] \ [[BR]] \ The 1.4.3 output is:[[BR]] \ [[BR]] \ display: none; width: 100px; height: 100px; left: 0px; top: 0px; position: absolute; opacity: 0; overflow: hidden; 0px 0px animation complete display: none; width: 420px; height: 150px; left: 38px; top: 97px; position: absolute; opacity: 1; 0px 0px[[BR]] \ [[BR]] \ And the Current Git Version output is:[[BR]] \ [[BR]] \ display: none; width: 100px; height: 100px; left: 0px; top: 0px; position: absolute; opacity: 0; overflow: hidden; 0px 0px animation complete display: none; width: 420px; height: 150px; left: 38px; top: 97px; position: absolute; opacity: 1; 0px 0px [[BR]] \ [[BR]] \ Here is solution for:[[BR]] \ jQuery JavaScript Library v1.5, string 7610, function jQuery.fx.prototype.update[[BR]] \ jQuery version > 1.4.3: the same mods.[[BR]] \ \ {{{ \ // Set display property to block for height/width animations \ if ( ( this.prop === "height" || this.prop === "width" ) && this.elem.style ) { \ this.elem.style.display = "block"; \ } \ \ }}} \ Please, make this changes next update.

Changed February 03, 2011 06:03PM UTC by jitter comment:2

component: unfiled → css
priority: undecided → low
resolution: → duplicate
status: new → closed

Changed February 03, 2011 06:03PM UTC by jitter comment:3

Duplicate of #7305.