Offset: account for scroll when calculating position

Richard McDaniel · timmywil · commit 0654711e0d92 · 2015-05-12T10:36:45.000-04:00
Fixes gh-1708 Close gh-1714
diff --git a/src/offset.js b/src/offset.js
@@ -137,8 +137,11 @@ jQuery.fn.extend({
 			}
 
 			// Add offsetParent borders
-			parentOffset.top  += jQuery.css( offsetParent[ 0 ], "borderTopWidth", true );
-			parentOffset.left += jQuery.css( offsetParent[ 0 ], "borderLeftWidth", true );
+			// Subtract offsetParent scroll positions
+			parentOffset.top += jQuery.css( offsetParent[ 0 ], "borderTopWidth", true ) -
+				offsetParent.scrollTop();
+			parentOffset.left += jQuery.css( offsetParent[ 0 ], "borderLeftWidth", true ) -
+				offsetParent.scrollLeft();
 		}
 
 		// Subtract parent offsets and element margins
diff --git a/test/unit/offset.js b/test/unit/offset.js
@@ -412,7 +412,7 @@ testIframe("offset/table", "table", function( $ ) {
 });
 
 testIframe("offset/scroll", "scroll", function( $, win ) {
-	expect(24);
+	expect(28);
 
 	// If we're going to bastardize the tests, let's just DO it
 	var ie = /msie 8/i.test( navigator.userAgent );
@@ -473,6 +473,17 @@ testIframe("offset/scroll", "scroll", function( $, win ) {
 	notEqual( $().scrollLeft(null), null, "jQuery().scrollLeft(null) testing setter on empty jquery object" );
 	strictEqual( $().scrollTop(), null, "jQuery().scrollTop(100) testing setter on empty jquery object" );
 	strictEqual( $().scrollLeft(), null, "jQuery().scrollLeft(100) testing setter on empty jquery object" );
+
+	// Tests position after parent scrolling (#15239)
+	$("#scroll-1").scrollTop(0);
+	$("#scroll-1").scrollLeft(0);
+	equal( $("#scroll-1-1").position().top, 6, "jQuery('#scroll-1-1').position().top unaffected by parent scrolling" );
+	equal( $("#scroll-1-1").position().left, 6, "jQuery('#scroll-1-1').position().left unaffected by parent scrolling" );
+
+	$("#scroll-1").scrollTop(5);
+	$("#scroll-1").scrollLeft(5);
+	equal( $("#scroll-1-1").position().top, 6, "jQuery('#scroll-1-1').position().top unaffected by parent scrolling" );
+	equal( $("#scroll-1-1").position().left, 6, "jQuery('#scroll-1-1').position().left unaffected by parent scrolling" );
 });
 
 testIframe("offset/body", "body", function( $ ) {

Original file line number	Diff line number	Diff line change
`@@ -137,8 +137,11 @@ jQuery.fn.extend({`
`137`	`137`	`}`
`138`	`138`
`139`	`139`	`// Add offsetParent borders`
`140`		`- parentOffset.top += jQuery.css( offsetParent[ 0 ], "borderTopWidth", true );`
`141`		`- parentOffset.left += jQuery.css( offsetParent[ 0 ], "borderLeftWidth", true );`
	`140`	`+ // Subtract offsetParent scroll positions`
	`141`	`+ parentOffset.top += jQuery.css( offsetParent[ 0 ], "borderTopWidth", true ) -`
	`142`	`+ offsetParent.scrollTop();`
	`143`	`+ parentOffset.left += jQuery.css( offsetParent[ 0 ], "borderLeftWidth", true ) -`
	`144`	`+ offsetParent.scrollLeft();`
`142`	`145`	`}`
`143`	`146`
`144`	`147`	`// Subtract parent offsets and element margins`