Use a for loop rather than for/in loop when copying events, so that code will work...

[jquery.git] / test / unit / offset.js

diff --git a/test/unit/offset.js b/test/unit/offset.js
index 284562e..8797531 100644 (file)
--- a/test/unit/offset.js
+++ b/test/unit/offset.js
@@ -1,5 +1,14 @@
 module("offset");
 
+test("disconnected node", function() {
+       expect(2);
+
+       var result = jQuery( document.createElement("div") ).offset();
+
+       equals( result.top, 0, "Check top" );
+       equals( result.left, 0, "Check left" );
+});
+
 var supportsScroll = false;
 
 testoffset("absolute"/* in iframe */, function($, iframe) {
@@ -43,7 +52,7 @@ testoffset("absolute"/* in iframe */, function($, iframe) {
 });
 
 testoffset("absolute", function( jQuery ) {
-       expect(144);
+       expect(178);
        
        // get offset tests
        var tests = [
@@ -70,6 +79,11 @@ testoffset("absolute", function( jQuery ) {
                equals( jQuery( this.id ).position().left, this.left, "jQuery('" + this.id + "').position().left" );
        });
        
+       // test #5781
+       var offset = jQuery( '#positionTest' ).offset({ top: 10, left: 10 }).offset();
+       equals( offset.top,  10, "Setting offset on element with position absolute but 'auto' values." )
+       equals( offset.left, 10, "Setting offset on element with position absolute but 'auto' values." )
+       
        
        // set offset
        tests = [
@@ -102,8 +116,14 @@ testoffset("absolute", function( jQuery ) {
                        equals( val.left, left, "Verify incoming top position." );
                        return { top: top + 1, left: left + 1 };
                });
-               equals( jQuery( this.id ).offset().top,  this.top + 1,  "jQuery('" + this.id + "').offset({ top: "  + this.top  + " })" );
-               equals( jQuery( this.id ).offset().left, this.left + 1, "jQuery('" + this.id + "').offset({ left: " + this.left + " })" );
+               equals( jQuery( this.id ).offset().top,  this.top  + 1, "jQuery('" + this.id + "').offset({ top: "  + (this.top  + 1) + " })" );
+               equals( jQuery( this.id ).offset().left, this.left + 1, "jQuery('" + this.id + "').offset({ left: " + (this.left + 1) + " })" );
+               
+               jQuery( this.id )
+                       .offset({ left: this.left + 2 })
+                       .offset({ top:  this.top  + 2 });
+               equals( jQuery( this.id ).offset().top,  this.top  + 2, "Setting one property at a time." );
+               equals( jQuery( this.id ).offset().left, this.left + 2, "Setting one property at a time." );
                
                jQuery( this.id ).offset({ top: this.top, left: this.left, using: function( props ) {
                        jQuery( this ).css({
@@ -120,7 +140,7 @@ testoffset("relative", function( jQuery ) {
        expect(60);
        
        // IE is collapsing the top margin of 1px
-       var ie = jQuery.browser.msie && parseInt( jQuery.browser.version ) < 8;
+       var ie = jQuery.browser.msie && parseInt( jQuery.browser.version, 10 ) < 8;
        
        // get offset
        var tests = [
@@ -181,7 +201,7 @@ testoffset("static", function( jQuery ) {
        expect(80);
        
        // IE is collapsing the top margin of 1px
-       var ie = jQuery.browser.msie && parseInt( jQuery.browser.version ) < 8;
+       var ie = jQuery.browser.msie && parseInt( jQuery.browser.version, 10 ) < 8;
        
        // get offset
        var tests = [
@@ -315,7 +335,7 @@ testoffset("table", function( jQuery ) {
 testoffset("scroll", function( jQuery, win ) {
        expect(16);
        
-       var ie = jQuery.browser.msie && parseInt( jQuery.browser.version ) < 8;
+       var ie = jQuery.browser.msie && parseInt( jQuery.browser.version, 10 ) < 8;
        
        // IE is collapsing the top margin of 1px
        equals( jQuery('#scroll-1').offset().top, ie ? 6 : 7, "jQuery('#scroll-1').offset().top" );
@@ -425,7 +445,7 @@ function testoffset(name, fn) {
        });
        
        function loadFixture() {
-               var src = './data/offset/' + name + '.html?' + parseInt( Math.random()*1000 ),
+               var src = './data/offset/' + name + '.html?' + parseInt( Math.random()*1000, 10 ),
                        iframe = jQuery('<iframe />').css({
                                width: 500, height: 500, position: 'absolute', top: -600, left: -600, visiblity: 'hidden'
                        }).appendTo('body')[0];