Bug Tracker

Changeset 5597

Show
Ignore:
Timestamp:
05/14/08 17:54:54 (8 months ago)
Author:
aflesler
Message:

jquery offset: reducing the code size.

Files:
1 modified

Legend:

Unmodified
Added
Removed
  • trunk/jquery/src/offset.js

    r5577 r5597  
    102102jQuery.fn.extend({ 
    103103    position: function() { 
    104         var left = 0, top = 0, elem = this[0], offset, parentOffset, offsetParent, results; 
     104        var left = 0, top = 0, results; 
    105105 
    106         if (elem) { 
     106        if ( this[0] ) { 
    107107            // Get *real* offsetParent 
    108             offsetParent = this.offsetParent(); 
     108            var offsetParent = this.offsetParent(), 
    109109 
    110             // Get correct offsets 
    111             offset       = this.offset(); 
    112             parentOffset = offsetParent.offset(); 
     110                // Get correct offsets 
     111                offset       = this.offset(),                
     112                parentOffset = offsetParent.offset(); 
    113113 
    114114            // Subtract element margins 
    115             offset.top  -= parseInt( jQuery.curCSS(elem, 'marginTop', true) ) || 0; 
    116             offset.left -= parseInt( jQuery.curCSS(elem, 'marginLeft', true) ) || 0; 
     115            offset.top  -= num( this, 'marginTop' ); 
     116            offset.left -= num( this, 'marginLeft' ); 
    117117 
    118118            // Add offsetParent borders 
    119             parentOffset.top  += parseInt( jQuery.curCSS(offsetParent[0], 'borderTopWidth', true) ) || 0; 
    120             parentOffset.left += parseInt( jQuery.curCSS(offsetParent[0], 'borderLeftWidth', true) ) || 0; 
     119            parentOffset.top  += num( offsetParent, 'borderTopWidth' ); 
     120            parentOffset.left += num( offsetParent, 'borderLeftWidth' ); 
    121121 
    122122            // Subtract the two offsets