Revert "Offset: allow offset setter to throw for disconnected elements"

timmywil · timmywil · commit ee69e9c8b8a5 · 2015-06-16T10:55:48.000-04:00
This reverts commit 0d11c11.
diff --git a/src/offset.js b/src/offset.js
@@ -39,7 +39,7 @@ jQuery.offset = {
 			elem.style.position = "relative";
 		}
 
-		curOffset = curElem.offset();
+		curOffset = curElem.offset() || { top: 0, left: 0 };
 		curCSSTop = jQuery.css( elem, "top" );
 		curCSSLeft = jQuery.css( elem, "left" );
 		calculatePosition = ( position === "absolute" || position === "fixed" ) &&
diff --git a/test/unit/core.js b/test/unit/core.js
@@ -36,6 +36,10 @@ test("jQuery()", function() {
 		expected++;
 		attrObj["width"] = 10;
 	}
+	if ( jQuery.fn.offset ) {
+		expected++;
+		attrObj["offset"] = { "top": 1, "left": 1 };
+	}
 	if ( jQuery.fn.css ) {
 		expected += 2;
 		attrObj["css"] = { "paddingLeft": 1, "paddingRight": 1 };
@@ -101,12 +105,16 @@ test("jQuery()", function() {
 	elem = jQuery("\n\n<em>world</em>")[0];
 	equal( elem.nodeName.toLowerCase(), "em", "leading newlines" );
 
-	elem = jQuery( "<div/>", attrObj );
+	elem = jQuery("<div/>", attrObj );
 
 	if ( jQuery.fn.width ) {
 		equal( elem[0].style.width, "10px", "jQuery() quick setter width");
 	}
 
+	if ( jQuery.fn.offset ) {
+		equal( elem[0].style.top, "1px", "jQuery() quick setter offset");
+	}
+
 	if ( jQuery.fn.css ) {
 		equal( elem[0].style.paddingLeft, "1px", "jQuery quick setter css");
 		equal( elem[0].style.paddingRight, "1px", "jQuery quick setter css");

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ jQuery.offset = {`
`39`	`39`	`elem.style.position = "relative";`
`40`	`40`	`}`
`41`	`41`
`42`		`- curOffset = curElem.offset();`
	`42`	`+ curOffset = curElem.offset() \|\| { top: 0, left: 0 };`
`43`	`43`	`curCSSTop = jQuery.css( elem, "top" );`
`44`	`44`	`curCSSLeft = jQuery.css( elem, "left" );`
`45`	`45`	`calculatePosition = ( position === "absolute" \|\| position === "fixed" ) &&`