Explicit overflow:auto is overridden by inline overflow:hidden during animation

[yonjah@…]

This might be two bugs or one (I'm not really sure).

jQuery will use the following method (line 1612) ->

// Check if elements with layout shrink-wrap their children
// (IE 6 does this)
div.style.display = "block";
div.style.overflow = "visible";
div.innerHTML = "<div style='width:5px;'></div>";
support.shrinkWrapBlocks = ( div.offsetWidth !== 3 );

unfortunately this will fail on new browsers if document style settings will set all elements to use "box-sizing:border-box" ( I only tested it in chrome 18 but will probably repeat it self with any border-box supported browsers)

I wouldn't have noticed this issue if jQuery animation wouldn't set my animated elements overflow to hidden and but then fail to remove them when animation ends (line 8892) ->

if ( options.overflow != null && !jQuery.support.shrinkWrapBlocks ) {

	jQuery.each( [ "", "X", "Y" ], function( index, value ) {
		elem.style[ "overflow" + value ] = options.overflow[ index ];
	});
}

I don't know what the original IE6 bug was that this flag tried to solve, but this seems to be the only place that it's used. and it would seem wierd that this is a desired action even on IE6 (chnaging original overflow settings and not restoring them when animation ends).

Yoni.

[rwaldron]

Thanks for taking the time to contribute to the jQuery project! Please provide a complete reduced test case on jsFiddle to help us assess your ticket!

Additionally, be sure to test against the "jQuery (edge)" version to ensure the issue still exists. To get you started, use this boilerplate:  http://jsfiddle.net/FrKyN/ Open the link and click to "Fork" (in the top menu) to begin.

[mikesherov]

this is resolved in jQuery 1.8

[yonjah@…]

Fiddle test code ->  http://jsfiddle.net/GEZ4B/

(this is using JQuery edge so I'm not sure if this is really resolved in 1.8)

[rwaldron]

Confirmed. Further reduced:

 http://jsfiddle.net/rwaldron/aXAE8/

[rwaldron]

Updating title to make sense

[scott.gonzalez]

rwaldron: jQuery edge is master, not 1.8pre.

[rwaldron]

Thanks scott. I tested this with 1.8pre  http://jsfiddle.net/rwaldron/aXAE8/ and there is no longer an issue

[mikesherov]

YOU'RE WELCOME

[rwaldron]

you're* :P

[yonjah@…]

just updated jQuery to v1.8.1 and this issue is still happening

[timmywil]

overflow: hidden is left there.

[elgreg]

#12456 is a duplicate of this ticket.

[petro]

Updated test case:  http://jsfiddle.net/mpetrovich/cpw9J/

[Mike Petrovich]

Fixes #11635, Explicit overflow:auto is overridden by inline overflow:hidden during animation, closes gh-981

Changeset: 861476eb3fb5e378fdbb4e86e4737daf6de27f13

[mikesherov]

#12630 is a duplicate of this ticket.