Problem with scrollto offset javascript because of header

Question

I need help with this scrollto code snippet. The problem is that I need to set an offset to account for my menu. Without the offset the header that I scroll to gets buried underneath the menu. See for yourselves here:https://julyfx.mystagingwebsite.com/stanford-mba-msx-essay-topic-analysis-examples/ Would anyone happen to have a suggestion? Thank you! Leah

<script type="text/javascript">
jQuery(document).ready(function($) {
   

$('a[href*="#"]')
  // Remove links that don't actually link to anything
  .not('[href="#"]')
  .not('[href="#0"]')
  .click(function(event) {
    // On-page links
    if (
      location.pathname.replace(/^\//, '') == this.pathname.replace(/^\//, '') 
      && 
      location.hostname == this.hostname
    ) {
      // Figure out element to scroll to
      var target = $(this.hash);
      target = target.length ? target : $('[name=' + this.hash.slice(1) + ']');
      // Does a scroll target exist?
      if (target.length) {
        // Only prevent default if animation is actually gonna happen
        event.preventDefault();
        $('html, body').animate({
          scrollTop: target.offset().top
        }, 1000, function() {
          // Callback after animation
          // Must change focus!
          var $target = $(target);
          $target.focus();
          if ($target.is(":focus")) { // Checking if the target was focused
            return false;
          } else {
            $target.attr('tabindex','-1'); // Adding tabindex for elements not focusable
            $target.focus(); // Set focus again
          };
        });
      }
    }
  });
});
</script>

Johannes · Accepted Answer · 2018-10-27 23:50:56Z

1

Should be relatively easy: If your header is for example 85px high, you can just add these 85px to the offset in your code, so this line

scrollTop: target.offset().top

becomes

scrollTop: target.offset().top - 85

that way the window will scroll 85px less than calculated, so the section will not be hidden behind the header.

answered Oct 27, 2018 at 23:50

Johannes

67.9k22 gold badges85 silver badges139 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

LCD Over a year ago

Thank you Johannes! You really helped me out!

Kummer Wolfe · Accepted Answer · 2018-10-27 23:46:27Z

0

Just a suggestion have you tried something like:

window.scrollTo({top: (jQuery('#2').position().top-jQuery('header').height()), behavior: 'smooth' })

?

Where #2 would be taken from your this.hash target above?

answered Oct 27, 2018 at 23:46

Kummer Wolfe

6035 silver badges13 bronze badges

2 Comments

LCD Over a year ago

Hi Kummer, Thanks for your suggestion. Unfortunately I'm completely unsophisticated when it comes to all this. I thought the solution might have something to do with adding a numeric offset around the target.offset() or something. But I'm not sure.

Kummer Wolfe Over a year ago

I understand. And yes, adding a numerical offset will do the trick. By getting jQuery('header').height() you could get such a number to offset your target.offset().top with. It would even account for the header changing height when on say a mobile device.

Collectives™ on Stack Overflow

Problem with scrollto offset javascript because of header

2 Answers 2

1 Comment

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related