Witam. Mam taki skrypt do rozsuwania menu w jquery

[JAVASCRIPT] pobierz, plaintext 
$(document).ready(function(){
 
$('ul#menu-pionowe li li').show().not('ul#menu-pionowe li.current_menu_parent li').hide()
 
         $("ul#menu-pionowe li").click(function() 
		 {    
			  $('li', this).toggle("normal");
 
			   if($(this).parent().attr('class') != "sub-menu")
			   {
				   alert('spelnia warunek');
					return false;
			   }else alert('nie spelnia warunku');
          });
return false;
});
[JAVASCRIPT] pobierz, plaintext 

Jeśli nie jest to class sub-menu to nie otwieraj linku i ten warunek działa. Ale jeśli jest to sub-menu to nie spełnia warunku i wykonuje sie else a nastepnie wykonuje sie spelnienie warunku bo to li jest wewnatrz li ktore je spelnia ... Dokładnie widać to tutaj w lewy menu ...

Użyj "hasClass"

Cześć.

Może problem tkwi w tym, że niema klamer?

Linia 13. za else jest alert() ale niema go w klamrach.


Zobacz także ten link


Pozdrawiam.

klamry są zbędne jeżeli chcemy wykonać tylko jedno polecenie.


Tak powinno wyglądać.

[JAVASCRIPT] pobierz, plaintext 
if($(this).parent().hasClass("sub-menu"))
			   {
				   alert('spelnia warunek');
			   }else alert('nie spelnia warunku');
[JAVASCRIPT] pobierz, plaintext 

Dlatego napisałem "może" gdyż nie byłem pewien czy jeżeli czynność z pierwszego warunku jest w klamrach to czy przy else kiedy druga czynność ma się wykonać też ma być w klamrach.