Commit 4dcd7dd3 authored by Mike Sherov's avatar Mike Sherov Committed by Rick Waldron

Fixe dimensions tests. No core code changes. Supplements #11856

parent e680f36c
...@@ -24,7 +24,7 @@ jQuery.each( { Height: "height", Width: "width" }, function( name, type ) { ...@@ -24,7 +24,7 @@ jQuery.each( { Height: "height", Width: "width" }, function( name, type ) {
doc = elem.documentElement; doc = elem.documentElement;
// Either scroll[Width/Height] or offset[Width/Height] or client[Width/Height], whichever is greatest // Either scroll[Width/Height] or offset[Width/Height] or client[Width/Height], whichever is greatest
// unfortunately, this causes bug #3838 in IE6 only, but there is currently no good, small way to fix it. // unfortunately, this causes bug #3838 in IE6/8 only, but there is currently no good, small way to fix it.
return Math.max( return Math.max(
elem.body[ scrollProp ], doc[ scrollProp ], elem.body[ scrollProp ], doc[ scrollProp ],
elem.body[ offsetProp ], doc[ offsetProp ], elem.body[ offsetProp ], doc[ offsetProp ],
......
...@@ -415,10 +415,16 @@ test("setters with and without box-sizing:border-box", function(){ ...@@ -415,10 +415,16 @@ test("setters with and without box-sizing:border-box", function(){
}); });
testIframe( "dimensions/documentSmall", "window vs. small document", function( jQuery, window, document ) { testIframe( "dimensions/documentSmall", "window vs. small document", function( jQuery, window, document ) {
expect(2); // this test is practically tautological, but there is a bug in IE8
// with no simple workaround, so this test exposes the bug and works around it
if ( document.body.offsetWidth >= document.documentElement.offsetWidth ) {
expect( 2 );
equal( jQuery( document ).height(), jQuery( window ).height(), "document height matches window height" ); equal( jQuery( document ).height(), jQuery( window ).height(), "document height matches window height" );
equal( jQuery( document ).width(), jQuery( window ).width(), "document width matches window width" ); equal( jQuery( document ).width(), jQuery( window ).width(), "document width matches window width" );
} else {
expect( 0 );
}
}); });
testIframe( "dimensions/documentLarge", "window vs. large document", function( jQuery, window, document ) { testIframe( "dimensions/documentLarge", "window vs. large document", function( jQuery, window, document ) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment