Offset: offsetLeft/Top on empty set returns undefined

Fixes gh-2319
Closes gh-2702
Ref gh-2701

diff --git a/src/offset.js b/src/offset.js
index bef2a8fa038ed5e0553561a3042cd97edb183ccd..f7ef79c3c7fa8853effc0afe53b9fe7632648bcc 100644 (file)
--- a/src/offset.js
+++ b/src/offset.js
@@ -202,7 +202,7 @@ jQuery.each( { scrollLeft: "pageXOffset", scrollTop: "pageYOffset" }, function(
                        } else {
                                elem[ method ] = val;
                        }
-               }, method, val, arguments.length, null );
+               }, method, val, arguments.length );
        };
 } );
 

diff --git a/test/unit/offset.js b/test/unit/offset.js
index 85f1da65ded7385a022e2fe8cc203e19b25a1841..a0f253b4e071f45ed6444296511d49cdf201d66b 100644 (file)
--- a/test/unit/offset.js
+++ b/test/unit/offset.js
@@ -460,8 +460,8 @@ testIframe( "offset/scroll", "scroll", function( $, win, doc, assert ) {
        assert.notEqual( $().scrollLeft( 100 ), null, "jQuery().scrollLeft(100) testing setter on empty jquery object" );
        assert.notEqual( $().scrollTop( null ), null, "jQuery().scrollTop(null) testing setter on empty jquery object" );
        assert.notEqual( $().scrollLeft( null ), null, "jQuery().scrollLeft(null) testing setter on empty jquery object" );
-       assert.strictEqual( $().scrollTop(), null, "jQuery().scrollTop(100) testing setter on empty jquery object" );
-       assert.strictEqual( $().scrollLeft(), null, "jQuery().scrollLeft(100) testing setter on empty jquery object" );
+       assert.strictEqual( $().scrollTop(), undefined, "jQuery().scrollTop() testing getter on empty jquery object" );
+       assert.strictEqual( $().scrollLeft(), undefined, "jQuery().scrollLeft() testing getter on empty jquery object" );
 
        // Tests position after parent scrolling (#15239)
        $( "#scroll-1" ).scrollTop( 0 );