Fix #13743. Remove deprecated support.boxModel. Close gh-1231.

add myself to AUTHORS.txt
jquery · Apr 18, 2013 · 35784a8 · 35784a8
1 parent 0db70aa
commit 35784a8
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 19 deletions.
diff --git a/AUTHORS.txt b/AUTHORS.txt
@@ -162,4 +162,5 @@ Jean Boussier <jean.boussier@gmail.com>
 Adam Coulombe <me@adam.co>
 Andrew Plummer <plummer.andrew@gmail.com>
 Dmitry Gusev <dmitry.gusev@gmail.com>
+Michał Gołębiowski <m.goleb@gmail.com>
 Brandon Johnson <bjohn465+github@gmail.com>
diff --git a/src/support.js b/src/support.js
@@ -66,9 +66,6 @@ jQuery.support = (function( support ) {
 	// Where outerHTML is undefined, this still works
 	support.html5Clone = document.createElement("nav").cloneNode( true ).outerHTML !== "<:nav></:nav>";
 
-	// jQuery.support.boxModel DEPRECATED in 1.8 since we don't support Quirks Mode
-	support.boxModel = document.compatMode === "CSS1Compat";
-
 	// Will be defined later
 	support.inlineBlockNeedsLayout = false;
 	support.shrinkWrapBlocks = false;

diff --git a/test/unit/support.js b/test/unit/support.js
@@ -1,11 +1,5 @@
 module("support", { teardown: moduleTeardown });
 
-test("boxModel", function() {
-	expect( 1 );
-
-	equal( jQuery.support.boxModel, document.compatMode === "CSS1Compat" , "jQuery.support.boxModel is sort of tied to quirks mode but unstable since 1.8" );
-});
-
 test( "zoom of doom (#13089)", function() {
 	expect( 1 );
 
@@ -76,7 +70,6 @@ testIframeWithCallback( "box-sizing does not affect jQuery.support.shrinkWrapBlo
 			"radioValue":true,
 			"checkClone":true,
 			"appendChecked":true,
-			"boxModel":true,
 			"reliableHiddenOffsets":true,
 			"ajax":true,
 			"cors":true,
@@ -110,7 +103,6 @@ testIframeWithCallback( "box-sizing does not affect jQuery.support.shrinkWrapBlo
 			"radioValue":false,
 			"checkClone":true,
 			"appendChecked":true,
-			"boxModel":true,
 			"reliableHiddenOffsets":true,
 			"ajax":true,
 			"cors":true,
@@ -144,7 +136,6 @@ testIframeWithCallback( "box-sizing does not affect jQuery.support.shrinkWrapBlo
 			"radioValue":false,
 			"checkClone":true,
 			"appendChecked":true,
-			"boxModel":true,
 			"reliableHiddenOffsets":true,
 			"ajax":true,
 			"cors":true,
@@ -178,7 +169,6 @@ testIframeWithCallback( "box-sizing does not affect jQuery.support.shrinkWrapBlo
 			"radioValue":false,
 			"checkClone":true,
 			"appendChecked":true,
-			"boxModel":true,
 			"reliableHiddenOffsets":true,
 			"ajax":true,
 			"cors":false,
@@ -212,7 +202,6 @@ testIframeWithCallback( "box-sizing does not affect jQuery.support.shrinkWrapBlo
 			"radioValue":false,
 			"checkClone":true,
 			"appendChecked":true,
-			"boxModel":true,
 			"reliableHiddenOffsets":false,
 			"ajax":true,
 			"cors":false,
@@ -223,7 +212,6 @@ testIframeWithCallback( "box-sizing does not affect jQuery.support.shrinkWrapBlo
 		expected = {
 			"ajax": true,
 			"appendChecked": false,
-			"boxModel": true,
 			"changeBubbles": false,
 			"checkClone": false,
 			"checkOn": true,
@@ -280,7 +268,6 @@ testIframeWithCallback( "box-sizing does not affect jQuery.support.shrinkWrapBlo
 			"radioValue":false,
 			"checkClone":false,
 			"appendChecked":false,
-			"boxModel":true,
 			"reliableHiddenOffsets":false,
 			"ajax":true,
 			"cors":false,
@@ -314,7 +301,6 @@ testIframeWithCallback( "box-sizing does not affect jQuery.support.shrinkWrapBlo
 			"radioValue":true,
 			"checkClone":false,
 			"appendChecked":false,
-			"boxModel":true,
 			"reliableHiddenOffsets":true,
 			"ajax":true,
 			"cors":true,
@@ -348,7 +334,6 @@ testIframeWithCallback( "box-sizing does not affect jQuery.support.shrinkWrapBlo
 			"radioValue":true,
 			"checkClone":true,
 			"appendChecked":true,
-			"boxModel":true,
 			"reliableHiddenOffsets":true,
 			"ajax":true,
 			"cors":true,
@@ -359,7 +344,7 @@ testIframeWithCallback( "box-sizing does not affect jQuery.support.shrinkWrapBlo
 
 	if ( expected ) {
 		test("Verify that the support tests resolve as expected per browser", function() {
-			expect( 31 );
+			expect( 30 );
 
 			for ( var i in expected ) {
 				if ( jQuery.ajax || i !== "ajax" && i !== "cors" ) {