jQuery по щелчку <a> не будет работать функция

Question

В настоящее время у меня есть <a> witin a <li> который при нажатии открывается подменю.

Теперь я хочу сделать так, чтобы вы нажмете на это <a>, когда у его родителя есть класс .open, он закрывается. Но он не срабатывает.

HTML-:

<li class="menu-item-has-children open"> 
    <a href="#">emerging market</a> 
    <ul class="sub-menu"> 
     [sub menu stuff] 
    </ul> 
</li> 

Итак, когда вы щелкаете .open a, он должен первым hidesibling ul .sub-menu, а затем removeClassopen из parent

JQuery

jQuery(".open a").click(
    function(){ 
     console.log('here!'); // this never seems to fire 
     jQuery(this).sibling(".sub-menu").hide(); 
     jQuery(this).parent().removeClass("open"); 
}); 

JS Скрипка, показывающая (рабочая) функция открытия, но функция нерабочим закрытия

http://jsfiddle.net/aa5brt5v/

Answer 1

Почему бы не использовать просто переключите список ребенка?

$(".menu-item-has-children").click(function() { 
 
    $(this).find(".sub-menu").toggle(); 
 
});

/* Default is to hide sub menu */ 
 
ul.sub-menu { 
 
    display: none; 
 
    background-color: orange; 
 
}

<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></script> 
 
<li class="menu-item-has-children"> <a href="#">emerging market</a> 
 
    <ul class="sub-menu"> 
 
    <li>[sub menu stuff]</li> 
 
    </ul> 
 
</li>

Answer 2

Попробуйте с помощью этого (делегирования событий):

$(".menu-item-has-children a").on("click", ".open", function() { 
console.log('here!'); 
}); 

Answer 3

Выпуска потому что события назначены на DOMReady. В то время нет элементов .open, поскольку вы добавляете этот класс при щелчке элемента, что хорошо после запуска DOMReady. Вместо этого вы можете использовать одно событие click для элемента. Вы можете также привести в порядок вашу логику и DOM Traversal немного слишком, попробуйте это:

$(".menu-item-has-children").click(function() { 
    if ($(this).hasClass('open')) { 
     $(this).removeClass("open").find(".sub-menu").hide(); 
    } else { 
     $(this).addClass("open").find(".sub-menu").show(); 
    } 
}); 

Updated fiddle