talho/openphin

Function offset has 64 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    offset: function(options, returnObject) {
        var x = 0, y = 0, sl = 0, st = 0,
            elem = this[0], parent = this[0], op, parPos, elemPos = $.css(elem, 'position'),
            mo = $.browser.mozilla, ie = $.browser.msie, sf = $.browser.safari, oa = $.browser.opera,
            absparent = false, relparent = false, 

Avoid deeply nested control flow statements.

Open

                        if (mo && parent != elem && $.css(parent, 'overflow') != 'visible') {
                            x += num(parent, 'borderLeftWidth');
                            y += num(parent, 'borderTopWidth');
                        }

Function handleOffsetReturn has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

var handleOffsetReturn = function(elem, options, x, y, sl, st) {

Avoid deeply nested control flow statements.

Open

                        if (options.scroll) {
                            // get scroll offsets
                            sl += parent.scrollLeft;
                            st += parent.scrollTop;
                        }

Consider simplifying this complex logical expression.

Open

                    if ( (mo && !absparent && elemPos != 'fixed') || 
                         (ie && elemPos == 'static' && !relparent) ) {
                        x += num(parent, 'borderLeftWidth');
                        y += num(parent, 'borderTopWidth');
                    }

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    height: function() {
        if ( this[0] == window )
            return self.innerHeight ||
                $.boxModel && document.documentElement.clientHeight || 
                document.body.clientHeight;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    width: function() {
        if ( this[0] == window )
            return self.innerWidth ||
                $.boxModel && document.documentElement.clientWidth ||
                document.body.clientWidth;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    innerWidth: function() {
        return this[0] == window || this[0] == document ?
            this.width() :
            this.is(':visible') ?
                this[0].offsetWidth - num(this, 'borderLeftWidth') - num(this, 'borderRightWidth') :

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    outerWidth: function() {
        return this[0] == window || this[0] == document ?
            this.width() :
            this.is(':visible') ?
                this[0].offsetWidth :

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    innerHeight: function() {
        return this[0] == window || this[0] == document ?
            this.height() :
            this.is(':visible') ?
                this[0].offsetHeight - num(this, 'borderTopWidth') - num(this, 'borderBottomWidth') :

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    outerHeight: function() {
        return this[0] == window || this[0] == document ?
            this.height() :
            this.is(':visible') ?
                this[0].offsetHeight :

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        if ( this[0] == window || this[0] == document )
            return self.pageYOffset ||
                $.boxModel && document.documentElement.scrollTop ||
                document.body.scrollTop;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        if ( this[0] == window || this[0] == document )
            return self.pageXOffset ||
                $.boxModel && document.documentElement.scrollLeft ||
                document.body.scrollLeft;

• Read up
  Read up