Всем привет!
В JS совсем юн, а использовать надо. Изучаю потихоньку его, но пишу свои простые скрипты пока на JQuery. Поэтому, не судите строго студента. У меня вопрос к более опытным и знающим. Работаю над сайтом.Задача стоит чтобы при скроллинге header изменял высоту и размер логотипа. Написал код:
// header scroll
$(window).scroll(function() {
if ($(this).scrollTop() > 1) {
$('.navigation').hide(),
$('.header').addClass('header_sticky'),
$('.header__logo').addClass('header__logo_sticky'),
$('.header__switch').show('drop', { direction: 'right' });
} else {
$('.navigation').show(),
$('.header').removeClass('header_sticky'),
$('.header__logo').removeClass('header__logo_sticky'),
$('.header__switch').hide('drop', { direction: 'right' });
}
}); // end header scroll
Вопрос такой,
во-первых: правильно ли (синтаксически) перечислять команды через запятую (Изначально через ";" было. Но я так сделал потому что ошибку искать начал. Ошибка:header дергался, небольшой зазор в 1px сверху начал появляться-исчезать. Начал отлаживать, мучался, дошёл до инструкции if else попробовал через запятую прописать. И вроде дергаться перестал)
и во-вторых: Как можно этот код упростить? мне кажется он слишком громоздкий и не совсем верный хоть и рабочий.
Был бы признателен за помощь)
P.s: ну и в идеале (извините за наглость, конешно) было бы интересно на ванильном JS этот код посмотреть
