Помогите с JS

Dram,

$(document).ready( fStartTabs );
$(window).on( 'resize', fStartTabs );


function fStartTabs() {

	if ( $(window).width() >= 1024 ) {

		if(!sTop3) {
			sTop3 = jQuery('#tabs-menu').offset().top;
		}
		if (document.documentElement.scrollTop > sTop3 || self.pageYOffset > sTop3) {
			jQuery('#tabs-menu').css("position", "fixed");
			jQuery('#tabs-menu').css("top", "0px");
			jQuery('#tabs-menu').css("margin-top", "0px");
			jQuery('#tabs-menu').css("z-index", "100");
			jQuery('#tabs-menu').css("max-width", "735px");
			jQuery('#tabs-menu .active').css("border-bottom-color", "#E86500");
			jQuery('#tabs-menu li').css("border-top-left-radius", "0px");
			jQuery('#tabs-menu li').css("border-top-right-radius", "0px");
		} else if (document.documentElement.scrollTop < sTop3 || self.pageYOffset < sTop3) {
			jQuery('#tabs-menu').css("position", "static");
		}
		if (self.pageYOffset + document.documentElement.clientHeight > document.documentElement.scrollHeight) {
			jQuery('#tabs-menu').css("position", "static");
		}

	} else {
		jQuery('#tabs-menu').css("position", "fixed");
		jQuery('#tabs-menu').css("bottom", "0px");
		jQuery('#tabs-menu').css("left", "0px");
		jQuery('#tabs-menu').css("z-index", "100");
		jQuery('#tabs-menu').css("width", "100%");
		jQuery('#tabs-menu .active').css("margin-bottom", "0px");
		jQuery('#tabs-menu li').css("font-size", "16px");
		jQuery('#tabs-menu').css("max-width", "100%");	
	}

});

вместо этих правил, лучше класс ставить addclass.

Именно так и надо.

Все ли скобки закрыты - че то ошибку выдает?

да, именно так, а не так:

jQuery('#tabs-menu').addClass('your_class');

#tabs-menu.your_class {
	position: fixed;
	top: 0px;
	margin-top: 0px;
	z-index: 100;
	max-width: 735px;
}
#tabs-menu.your_class .active { border-bottom-color: #E86500; }
#tabs-menu.your_class { border-top-left-radius: 0px; border-top-right-radius: 0px; }

или так:

jQuery('#tabs-menu').css({"position": "fixed", "top": 0, "margin-top": 0, "z-index": 100, "max-width": 735});

Sitealert, надо именно минимум 5 раз находит один и тот селектор в DOM и столько же раз вызывать один и тот же метод. :)

---------- Добавлено 14.07.2017 в 12:22 ----------

в конце ошибка, вместо }) , должно быть }.

Вот мой код полностью, все равно ошибку выдает, что не так?

var sTop, sTop2, sTop3;

jQuery(document).ready(function() {
jQuery(document).scroll(function() {
// A
if(!sTop) {
sTop = jQuery('#advleft').offset().top + 100;
}
if (document.documentElement.scrollTop > sTop || self.pageYOffset > sTop) {
jQuery('#advleft').css("position", "fixed");
jQuery('#advleft').css("top", "0px");
} else if (document.documentElement.scrollTop < sTop || self.pageYOffset < sTop) {
jQuery('#advleft').css("position", "static");
}
if (self.pageYOffset + document.documentElement.clientHeight > document.documentElement.scrollHeight) {
jQuery('#advleft').css("position", "static");
}
// B
if(!sTop2) {
sTop2 = jQuery('#advright').offset().top + 100;
}
if (document.documentElement.scrollTop > sTop2 || self.pageYOffset > sTop2) {
jQuery('#advright').css("position", "fixed");
jQuery('#advright').css("top", "0px");
} else if (document.documentElement.scrollTop < sTop2 || self.pageYOffset < sTop2) {
jQuery('#advright').css("position", "static");
}
if (self.pageYOffset + document.documentElement.clientHeight > document.documentElement.scrollHeight) {
jQuery('#advright').css("position", "static");
}




if(!sTop3) {
$(document).ready( fStartTabs );
$(window).on( 'resize', fStartTabs );
}

function fStartTabs() {

if ( $(window).width() >= 1024 ) {

if(!sTop3) {
sTop3 = jQuery('#tabs-menu').offset().top;
}
if (document.documentElement.scrollTop > sTop3 || self.pageYOffset > sTop3) {
jQuery('#tabs-menu').css("position", "fixed");
jQuery('#tabs-menu').css("top", "0px");
jQuery('#tabs-menu').css("margin-top", "0px");
jQuery('#tabs-menu').css("z-index", "100");
jQuery('#tabs-menu').css("max-width", "735px");
jQuery('#tabs-menu .active').css("border-bottom-color", "#E86500");
jQuery('#tabs-menu li').css("border-top-left-radius", "0px");
jQuery('#tabs-menu li').css("border-top-right-radius", "0px");
} else if (document.documentElement.scrollTop < sTop3 || self.pageYOffset < sTop3) {
jQuery('#tabs-menu').css("position", "static");
}
if (self.pageYOffset + document.documentElement.clientHeight > document.documentElement.scrollHeight) {
jQuery('#tabs-menu').css("position", "static");
}

} else {
jQuery('#tabs-menu').css("position", "fixed");
jQuery('#tabs-menu').css("bottom", "0px");
jQuery('#tabs-menu').css("left", "0px");
jQuery('#tabs-menu').css("z-index", "100");
jQuery('#tabs-menu').css("width", "100%");
jQuery('#tabs-menu .active').css("margin-bottom", "0px");
jQuery('#tabs-menu li').css("font-size", "16px");
jQuery('#tabs-menu').css("max-width", "100%"); 
}

};




});
});

Если вкратце, то в этом условии if(!sTop3) закрывающую фигурную скобку пропустили.

dma84, к сожалению на дескопах меню не плавает, вообще никакой реакции.

Вот с таким кодом плавает, но с мобилами косячно