Hallo mely > Ich schäme mich eigentlich nochmal zu fragen, aber die Zeile > > ~~~ javascript > '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: > > ~~~ javascript > // ... > > $('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)`{: .language-javascript} ≠ `$target`{: .language-javascript} ~~~ javascript 'scrollTop' : $(target).offset().top // → soll es heißen! ;-) ~~~ Gruß, Orlok

Hallo mely > Ich schäme mich eigentlich nochmal zu fragen, aber die Zeile > > ~~~ javascript > '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: > > ~~~ javascript > // ... > > $('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)`{: .language-javascript} ≠ `$target`{: .language-javascript} ~~~ javascript 'scrollTop' : $(target).offset().top // → soll es heißen! ;-) ~~~ Gruß, Orlok

Hallo mely > Ich schäme mich eigentlich nochmal zu fragen, aber die Zeile > > ~~~ javascript > '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: > > ~~~ javascript > // ... > > $('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)`{: .language-javascript} ≠ `$target`{: .language-javascript} ~~~ javascript 'scrollTop' : $(target).offset().top // → soll es heißen! ;-) ~~~ Gruß, Orlok