Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixes affix-top class not applying #15154

Closed
wants to merge 2 commits into from
Closed

Fixes affix-top class not applying #15154

wants to merge 2 commits into from

Commits on Nov 16, 2014

  1. Fixes affix-top class not applying 

    Use scrollTop instead of colliderTop which uses the elements
offset().top, as the offset top does not account for padding.

This issue can be replicated by using a navbar-fixed-top and applying
relevant padding to the body. (A navbar-static-top with no padding on
the body does not encounter this issue)

Fixes #15078
    Nick O'Donoghue committed Nov 16, 2014
    Configuration menu
    Copy the full SHA
    a3c8a57 View commit details
    Browse the repository at this point in the history

Commits on Nov 17, 2014

  1. add unit test for affix-top for padding issue

    Nick O'Donoghue committed Nov 17, 2014
    Configuration menu
    Copy the full SHA
    9badfea View commit details
    Browse the repository at this point in the history