Commit c4e22ad8 authored by Mike Sherov's avatar Mike Sherov Committed by Dave Methvin

Fix #11721. Remove jQuery.boxModel, deprecate jQuery.support.boxModel.

This removes all internal uses of `jQuery.support.boxModel`. jQuery has never run unit tests with Quirks Mode and has not even feigned support for several years, so these remnants weren't doing much except giving false hope.

For now, `jQuery.support.boxModel` continues to have a value indicating whether the W3C box model is *generally* in use, but be aware that this is easily overridden on an element-by-element basis by the `box-model` CSS property. So don't trust this value.
parent f76518e3
...@@ -681,7 +681,7 @@ function defaultDisplay( nodeName ) { ...@@ -681,7 +681,7 @@ function defaultDisplay( nodeName ) {
// document to it; WebKit & Firefox won't allow reusing the iframe document. // document to it; WebKit & Firefox won't allow reusing the iframe document.
if ( !iframeDoc || !iframe.createElement ) { if ( !iframeDoc || !iframe.createElement ) {
iframeDoc = ( iframe.contentWindow || iframe.contentDocument ).document; iframeDoc = ( iframe.contentWindow || iframe.contentDocument ).document;
iframeDoc.write( ( jQuery.support.boxModel ? "<!doctype html>" : "" ) + "<html><body>" ); iframeDoc.write("<!doctype html><html><body>");
iframeDoc.close(); iframeDoc.close();
} }
......
...@@ -20,8 +20,8 @@ if ( "getBoundingClientRect" in document.documentElement ) { ...@@ -20,8 +20,8 @@ if ( "getBoundingClientRect" in document.documentElement ) {
win = getWindow( doc ), win = getWindow( doc ),
clientTop = docElem.clientTop || body.clientTop || 0, clientTop = docElem.clientTop || body.clientTop || 0,
clientLeft = docElem.clientLeft || body.clientLeft || 0, clientLeft = docElem.clientLeft || body.clientLeft || 0,
scrollTop = win.pageYOffset || jQuery.support.boxModel && docElem.scrollTop || body.scrollTop, scrollTop = win.pageYOffset || docElem.scrollTop,
scrollLeft = win.pageXOffset || jQuery.support.boxModel && docElem.scrollLeft || body.scrollLeft, scrollLeft = win.pageXOffset || docElem.scrollLeft,
top = box.top + scrollTop - clientTop, top = box.top + scrollTop - clientTop,
left = box.left + scrollLeft - clientLeft; left = box.left + scrollLeft - clientLeft;
...@@ -176,8 +176,7 @@ jQuery.each( {scrollLeft: "pageXOffset", scrollTop: "pageYOffset"}, function( me ...@@ -176,8 +176,7 @@ jQuery.each( {scrollLeft: "pageXOffset", scrollTop: "pageYOffset"}, function( me
if ( val === undefined ) { if ( val === undefined ) {
return win ? (prop in win) ? win[ prop ] : return win ? (prop in win) ? win[ prop ] :
jQuery.support.boxModel && win.document.documentElement[ method ] || win.document.documentElement[ method ] :
win.document.body[ method ] :
elem[ method ]; elem[ method ];
} }
......
...@@ -95,8 +95,8 @@ jQuery.support = (function() { ...@@ -95,8 +95,8 @@ jQuery.support = (function() {
boxSizingReliable: true boxSizingReliable: true
}; };
// jQuery.boxModel DEPRECATED in 1.3, use jQuery.support.boxModel instead // jQuery.support.boxModel DEPRECATED in 1.8 since we don't support Quirks Mode
jQuery.boxModel = support.boxModel = (document.compatMode === "CSS1Compat"); support.boxModel = (document.compatMode === "CSS1Compat");
// Make sure checked status is properly cloned // Make sure checked status is properly cloned
input.checked = true; input.checked = true;
......
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<body>
<script src="../include_js.php"></script>
<script>
jQuery(function() { window.parent.iframeCallback( document.compatMode, jQuery.support.boxModel ) });
</script>
</body>
</html>
...@@ -342,7 +342,7 @@ testIframe("offset/table", "table", function( jQuery ) { ...@@ -342,7 +342,7 @@ testIframe("offset/table", "table", function( jQuery ) {
}); });
testIframe("offset/scroll", "scroll", function( jQuery, win ) { testIframe("offset/scroll", "scroll", function( jQuery, win ) {
expect(24); expect(26);
var ie = jQuery.browser.msie && parseInt( jQuery.browser.version, 10 ) < 8; var ie = jQuery.browser.msie && parseInt( jQuery.browser.version, 10 ) < 8;
...@@ -397,6 +397,12 @@ testIframe("offset/scroll", "scroll", function( jQuery, win ) { ...@@ -397,6 +397,12 @@ testIframe("offset/scroll", "scroll", function( jQuery, win ) {
notEqual( jQuery().scrollLeft(null), null, "jQuery().scrollLeft(null) testing setter on empty jquery object" ); notEqual( jQuery().scrollLeft(null), null, "jQuery().scrollLeft(null) testing setter on empty jquery object" );
strictEqual( jQuery().scrollTop(), null, "jQuery().scrollTop(100) testing setter on empty jquery object" ); strictEqual( jQuery().scrollTop(), null, "jQuery().scrollTop(100) testing setter on empty jquery object" );
strictEqual( jQuery().scrollLeft(), null, "jQuery().scrollLeft(100) testing setter on empty jquery object" ); strictEqual( jQuery().scrollLeft(), null, "jQuery().scrollLeft(100) testing setter on empty jquery object" );
// test setting scroll
jQuery( win ).scrollTop( 100 );
jQuery( win ).scrollLeft( 101 );
equal( jQuery( win ).scrollTop(), 100, "jQuery( win ).scrollTop() testing setter" );
equal( jQuery( win ).scrollLeft(), 101, "jQuery( win ).scrollLeft() testing setter" );
}); });
testIframe("offset/body", "body", function( jQuery ) { testIframe("offset/body", "body", function( jQuery ) {
......
module("support", { teardown: moduleTeardown }); module("support", { teardown: moduleTeardown });
testIframeWithCallback( "proper boxModel in compatMode CSS1Compat (IE6 and IE7)", "support/boxModelIE", function( compatMode, boxModel ) { ok( jQuery.support.boxModel, "jQuery.support.boxModel is perpetually true since 1.8" );
ok( compatMode !== "CSS1Compat" || boxModel, "boxModel properly detected" );
});
testIframeWithCallback( "body background is not lost if set prior to loading jQuery (#9238)", "support/bodyBackground", function( color, support ) { testIframeWithCallback( "body background is not lost if set prior to loading jQuery (#9238)", "support/bodyBackground", function( color, support ) {
expect( 2 ); expect( 2 );
......
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