Offset: getBounding doesn't return width/height in IE8. Fixes test.

jquery · Jun 16, 2015 · 3b1de11 · 3b1de11
1 parent 523de77
commit 3b1de11
Showing 1 changed file with 7 additions and 14 deletions.
diff --git a/src/offset.js b/src/offset.js
@@ -108,21 +108,14 @@ jQuery.fn.extend({
 		}
 
 		rect = elem.getBoundingClientRect();
+		win = getWindow( doc );
 
-		// Make sure element is not hidden (display: none)
-		if ( rect.width || rect.height ) {
-			win = getWindow( doc );
-
-			return {
-				top: rect.top  + ( win.pageYOffset || docElem.scrollTop ) -
-					( docElem.clientTop  || 0 ),
-				left: rect.left + ( win.pageXOffset || docElem.scrollLeft ) -
-					( docElem.clientLeft || 0 )
-			};
-		}
-
-		// Return zeros for hidden elements
-		return rect;
+		return {
+			top: rect.top  + ( win.pageYOffset || docElem.scrollTop ) -
+				( docElem.clientTop  || 0 ),
+			left: rect.left + ( win.pageXOffset || docElem.scrollLeft ) -
+				( docElem.clientLeft || 0 )
+		};
 	},
 
 	position: function() {