Почему ругается на offset? (Cannot read property 'top' of undefined)?

Question

[serega__koval1503]

$(function(){
    funcWidth();
});
$(window).resize(function(){
    funcWidth();
});
function funcWidth(){
    var windowWidth = $(window).width();
    console.log(windowWidth);
    if(windowWidth <= 991) {
        $(".menu__navigation > li > a").on('click',function(e){
            e.preventDefault();
        });
    } else {
        $("[data-scroll-nav] a").on('click', function(e) {
            //e.preventDefault();
            var scrollTo = $(this).attr('href');
            $("body").animate({
                scrollTop: $(scrollTo).offset().top
            },500);
        });
    }
}

Comments

[OVK2015]

var scrollTo = $(this).attr('href');
scrollTop: $(scrollTo).offset().top
Потому что у обычной текстовой строки нет св-ва offset

[serega__koval1503]

OVK2015: Подскажите как это исправить?

[OVK2015]

serega__koval1503: Исправить что? До этой ссылки прокрутить?

[serega__koval1503]

OVK2015: ну Убрать эту ошибку,оно сначала крутится все нормально, когда меняю resize окна то,появляется эта ошибка и уже не работает(

[OVK2015]

serega__koval1503: Уже ответили. Хотя e.preventDefault(); там то же нужен, по идее

Answer 1

[Ivanq]

В jQuery не силен, но разве у $(строка) будет offset? Вызвать offset() нужно на this:
scrollTop: $(this).offset().top
(примерно так).
А href вообще не нужен.

Comments

[serega__koval1503]

Ivanq А как тогда мне по якорям передвигаться?

[Ivanq]

serega__koval1503: Так в scrollTop нужно количество пикселей. Насколько я понимаю, $("#...") не сработает. Лучше якорь вставить как <span/div/a/... id="блаблабла">, а потом scrollTop: $("#блаблабла").offset().top()