Hallo mely
Ich schäme mich eigentlich nochmal zu fragen, aber die Zeile
'scrollTop' : $(target).offset( ).top
ist/war genau so und habe da nichts verändert.
Schämen musst du dich für überhaupt nichts! Aber schau doch noch mal genau hin. :-)
Das ganze sieht so aus:
// ... $('nav a[href^="#"]').on('click', function (e) { e.preventDefault(); $(document).off("scroll"); $('nav a').each(function () { $(this).removeClass('active'); $("nav").removeClass("navTest"); }) // Semikolon vergessen → besser }); als nur }) $(this).addClass('active'); var target = this.hash; $('html, body').stop().animate({ 'scrollTop': $target.offset().top // Klammern vergessen → $(target).offset().top }, 500, 'swing', function () { window.location.hash = target; $(document).on("scroll", onScroll); }); }); // ...
Habe den Code mal ein wenig formatiert und die unwichtigen Teile rausgenommen.
Du hast bei der Zuweisung die Klammern vergessen: $(target)
≠ $target
'scrollTop' : $(target).offset().top // → soll es heißen! ;-)
Gruß,
Orlok