Skip to content
This repository has been archived by the owner on Jun 18, 2022. It is now read-only.

fix Issue #52 #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix Issue #52 #53

wants to merge 1 commit into from

Conversation

wweggplant
Copy link

My jQuery version is 1.8.
I find that if div.stuckMenu next element is empty,currentMarginT is NaN

 currentMarginT = parseInt($(this).closest('div').css('margin-top')) // line 64

it causes a problem that code does not work in line 100 ~ 103,

$('.stuckMenu').next().closest('div').css({
    'margin-top': stickyHeight + stickyMarginB + currentMarginT + 'px'
}, 10);

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant