В чем ошибка в синтакисисе JS?

Question

[Михаил Гильмутдинов]

if ($(window).width() < 991) {

	$(window).load(function() {

		$('.nav_slide_button').click(function() {
			$('.pull').slideToggle();
		});

	});

else $(".pull".css("display", "block");

}

Answer 1

[Aleksandr]

else $(".pull".css("display", "block");
Скобки нету

if ($(window).width() < 991) {
	$(window).load(function() {
		$('.nav_slide_button').click(function(event) {
			$('.pull').slideToggle();
		});
	});
}else{
	$(".pull").css("display", "block");
}

Comments

[Михаил Гильмутдинов]

Подскажи а как сделать чтобы это условие выполнялось моментально. Просто когда я изменяю ширину браузера ничего не меняется. Но когда изменяю а потом обновляю то меняется!

[Aleksandr]

Запихнуть в функцию resize

[Aleksandr]

$(window).resize(function() {
//function
});

[Михаил Гильмутдинов]

т.е. обернуть весь код в
$(window).resize(function() {
сюда поместить мой код?
});

[Михаил Гильмутдинов]

$(window).resize(function() {
//function

($(window).width() < 991||$(".pull").css("display", "block"))
&&$(window).load(function() {
$('.nav_slide_button').click(function(event) {
$('.pull').slideToggle();
});
});
});

но теперь часть кода не работает! $('.nav_slide_button').click(function(event) {
$('.pull').slideToggle();
});

Answer 2

[Виталий Инчин ☢]

//Пишут либо
if(){   }else{   }
//Либо 
if()     else

И после .pull скобку не закрыли.

//А я пишу так :)
($(window).width() < 991||!$(".pull").css("display", "block"))
    &&$(window).load(function() { 
        $('.nav_slide_button').click(function(event) {
            $('.pull').slideToggle();
        }); 
    });

Comments

[Pretor DH]

ТОлько перед else а не после.

[Pretor DH]

Но можно убрать после условия открывающуюся и тогда вообще не ставить закрывающуюся.

[Виталий Инчин ☢]

Pretor DH: да, конечно, как кому нравится. Я вод предпочитаю через опеераторы И, Или, если ситуация позволяет.

[Михаил Гильмутдинов]

Подскажите а как сделать чтобы это условие выполнялось моментально. Просто когда я изменяю ширину браузера ничего не меняется. Но когда изменяю а потом обновляю то меняется!

[Виталий Инчин ☢]

Михаил Гильмутдинов: load - это событие загрузки документа. Вам нужно resize

[Pretor DH]

Только код после && сработает полюбому, ведь $(".pull").css("display", "block") вернет объект.
Надо было бы писать ($(window).width() < 991||($(".pull").css("display", "block"),0)) && ...
или блок после || перенести в самый конец.

[Виталий Инчин ☢]

Pretor DH: ок, нне знал, тогда скорее даже !$().css

[Pretor DH]

!$().css() можно! Но я предпочитаю явно указать, потом легче анализировать код.

[Pretor DH]

Пробегая глазами !$()... все равно нужно дойти до конца условия и потом вернутся в начало удостоверится. А с 0 или 1 в конце сразу поняттно, что конец или продолжение.

[Виталий Инчин ☢]

Pretor DH: возьму такую запись на вооружение =3

Answer 3

[Денис Инешин]

Забыли } перед else и { после:

if (...) {
    // 1
} else {
    // 2
}

И вообще, вы в каком редакторе пишете этот код? Нормальный редактор сразу подсветит синтаксис