From 626cef9fa40a3759753383ff61d5ebde32bd9a18 Mon Sep 17 00:00:00 2001 From: "Aleksey V. Zapparov" Date: Mon, 2 Sep 2013 03:24:27 +0200 Subject: [PATCH] Pass $element to offset top/bottom calc funcs At the moment to make dynamic top offset calculator on multiple elements, one would write: ``` javascript $('.my-affix').each(function () { var $el = $(this); $el.affix({ offset: { top: function () { return $el.offset().top; } } }); }) ``` This patch will allow to: ``` javascript $('.my-affix').affix({ offset: { top: function ($el) { return $el.offset().top; } } }); ``` --- js/affix.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/js/affix.js b/js/affix.js index c7be96e1dd..84760779d5 100644 --- a/js/affix.js +++ b/js/affix.js @@ -57,8 +57,8 @@ var offsetBottom = offset.bottom if (typeof offset != 'object') offsetBottom = offsetTop = offset - if (typeof offsetTop == 'function') offsetTop = offset.top() - if (typeof offsetBottom == 'function') offsetBottom = offset.bottom() + if (typeof offsetTop == 'function') offsetTop = offset.top(this.$element) + if (typeof offsetBottom == 'function') offsetBottom = offset.bottom(this.$element) var affix = this.unpin != null && (scrollTop + this.unpin <= position.top) ? false : offsetBottom != null && (position.top + this.$element.height() >= scrollHeight - offsetBottom) ? 'bottom' :