From eff9b063da87c940d627745fe218a6c5435b69c1 Mon Sep 17 00:00:00 2001 From: fat Date: Mon, 21 Jul 2014 22:31:56 -0700 Subject: [PATCH] cleanup style --- js/affix.js | 41 +++++++++++++++++------------------------ 1 file changed, 17 insertions(+), 24 deletions(-) diff --git a/js/affix.js b/js/affix.js index 7f80041d5d..bd48fc2ca3 100644 --- a/js/affix.js +++ b/js/affix.js @@ -38,32 +38,25 @@ } Affix.prototype.getState = function (scrollHeight, height, offsetTop, offsetBottom) { - var scrollTop = this.$target.scrollTop() - var position = this.$element.offset() - var targetHeight = this.$target.height() + var scrollTop = this.$target.scrollTop() + var position = this.$element.offset() + var targetHeight = this.$target.height() + + if (offsetTop != null && this.affixed == 'top') return scrollTop < offsetTop ? 'top' : false - if (offsetTop != null && this.affixed == 'top') return scrollTop >= offsetTop ? false : 'top' if (this.affixed == 'bottom') { - // Can be affixed to the top, use the unpin value - if (offsetTop != null) { - return (scrollTop + this.unpin <= position.top) ? false : 'bottom' - // Can only ever be pinned or affixed to the bottom, ignore unpin value - } else { - return (scrollTop + targetHeight <= scrollHeight - offsetBottom) ? false : 'bottom' - } - } else { - var initializing = this.affixed == null - var colliderTop = initializing ? scrollTop : position.top - var colliderHeight = initializing ? targetHeight : height - - if (offsetTop != null && colliderTop <= offsetTop) { - return 'top' - } else if (offsetBottom != null && (colliderTop + colliderHeight >= scrollHeight - offsetBottom)) { - return 'bottom' - } else { - return false - } + if (offsetTop != null) return (scrollTop + this.unpin <= position.top) ? false : 'bottom' + return (scrollTop + targetHeight <= scrollHeight - offsetBottom) ? false : 'bottom' } + + var initializing = this.affixed == null + var colliderTop = initializing ? scrollTop : position.top + var colliderHeight = initializing ? targetHeight : height + + if (offsetTop != null && colliderTop <= offsetTop) return 'top' + if (offsetBottom != null && (colliderTop + colliderHeight >= scrollHeight - offsetBottom)) return 'bottom' + + return false } Affix.prototype.getPinnedOffset = function () { @@ -81,11 +74,11 @@ Affix.prototype.checkPosition = function () { if (!this.$element.is(':visible')) return - var scrollHeight = $('body').height() var height = this.$element.height() var offset = this.options.offset var offsetTop = offset.top var offsetBottom = offset.bottom + var scrollHeight = $('body').height() if (typeof offset != 'object') offsetBottom = offsetTop = offset if (typeof offsetTop == 'function') offsetTop = offset.top(this.$element)