Bug Tracker

Changeset 5598

Show
Ignore:
Timestamp:
05/14/08 18:06:59 (7 months ago)
Author:
aflesler
Message:

jquery offset: reducing more the code size.

Files:
1 modified

Legend:

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

    r5597 r5598  
    141141// Create scrollLeft and scrollTop methods 
    142142jQuery.each( ['Left', 'Top'], function(i, name) { 
    143     jQuery.fn[ 'scroll' + name ] = function(val) { 
     143    var method = 'scroll' + name; 
     144     
     145    jQuery.fn[ method ] = function(val) { 
    144146        if (!this[0]) return; 
    145147 
     
    150152                this == window || this == document ? 
    151153                    window.scrollTo( 
    152                         name == 'Left' ? val : jQuery(window)[ 'scrollLeft' ](), 
    153                         name == 'Top'  ? val : jQuery(window)[ 'scrollTop'  ]() 
     154                        !i ? val : jQuery(window).scrollLeft(), 
     155                        i ? val : jQuery(window).scrollTop() 
    154156                    ) : 
    155                     this[ 'scroll' + name ] = val; 
     157                    this[ method ] = val; 
    156158            }) : 
    157159 
    158160            // Return the scroll offset 
    159161            this[0] == window || this[0] == document ? 
    160                 self[ (name == 'Left' ? 'pageXOffset' : 'pageYOffset') ] || 
    161                     jQuery.boxModel && document.documentElement[ 'scroll' + name ] || 
    162                     document.body[ 'scroll' + name ] : 
    163                 this[0][ 'scroll' + name ]; 
     162                self[ i ? 'pageYOffset' : 'pageXOffset' ] || 
     163                    jQuery.boxModel && document.documentElement[ method ] || 
     164                    document.body[ method ] : 
     165                this[0][ method ]; 
    164166    }; 
    165167});