nginx-theme scroll sticky-menu fix

README.md

Hey, nginx.com

Your sticky-menu was broked, so I fixed it.

CSS

Problem:

#masthead is getting yanked from the layout and everything after it gets forced up to fill the 70px gap.

Solution:

Never put it in the layout flow to begin with. Use absolute positioning and recover the lost 70px with padding.

JS

Problem:

Looks like a copy + paste accident from the lines above them. If you scroll to "< 150", you are already "< 200" so both conditions fire at once almost.

Solution:

Check for the bigger number first.

nginx-theme.css

#masthead {
  position: absolute;
}
#masthead + div {
  padding-top: 70px;
}

nginx-theme.js

/*...*/
function scroll_effect() {
  /*...line 174...*/
  if (currentScrollValue < 200) {
    /*...*/

    if (currentScrollValue < 150) {
      /*...*/
    }
  }
}

PS - looks great when it works the way you intended it to :)