From 1ac2b4c793b23dcd5ba60b6e8e4a25b815740586 Mon Sep 17 00:00:00 2001 From: Aleksey Razbakov Date: Fri, 20 Jan 2017 11:08:30 +0100 Subject: [PATCH] change stickyElementHeight to include margins --- sticky-header.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sticky-header.js b/sticky-header.js index ccd152d..44ab16d 100644 --- a/sticky-header.js +++ b/sticky-header.js @@ -27,7 +27,9 @@ var jQuery = require('jquery'); var targetElement = settings.targetElement || this; var minTop = settings.minTop || $(targetElement).offset().top; - var stickyElementHeight = $(targetElement).height(); + var stickyElementHeight = $(settings.targetElement).height() + + parseInt($(settings.targetElement).css('margin-bottom').replace('px', '')) + + parseInt($(settings.targetElement).css('margin-top').replace('px', '')); $(window).scroll(function (event) { if (settings.minimalViewportWidth == 0 || verge.viewportW() < settings.minimalViewportWidth) {