change stickyElementHeight to include margins

kirchbergerknorr · Jan 20, 2017 · 1ac2b4c · 1ac2b4c
1 parent b603764
commit 1ac2b4c
Showing 1 changed file with 3 additions and 1 deletion.
diff --git 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) {