on («щелчок», событие) стрельба без нажатия

Question

Есть похожие вопросы по этой теме, но их ответы не работали для моей проблемы.

Im пытается использовать jQuery's on("click") для запуска функции, которая просто изменяет класс идентификатора, который был нажат. Но функция вызывается, когда страница загружается, а затем, если вы нажмете на то, что должно запускать функцию, которой она не пользуется.

Вот HTML:

<nav> 
    <ul> 
     <li id="all" class="tab-current"><a><span>All Departments</span></a></li> 
    </ul> 
</nav> 

А вот ЯШ:

var $all = $("#all"); 
$all.on("click", function(){ 
    tabClicked("all"); 
}); 

Я также попытался это:

$all.on("click", tabClicked("all")); 

И это функция:

function tabClicked(input){ 
    //do some stuff here 
} 

Любые идеи? Заранее спасибо! :)

Answer 1

var $all = $("#all"); 

$all.on("click", function(){ 
    tabClicked("all"); 
}); 

Попробуйте отредактировать его вот так. $(function() {}); это как $(document).ready(), поэтому поставьте function(){} вместо $(function() {});.

EDIT:

По какой-то причине, он стреляет, если script тег после элементов в HTML. Я пробовал ваш код с внешним JS и т. Д. - не работал.

Try, как это: JSFiddle

EDIT 2: Вы должны поставить свой обработчик в $(document).ready() функции в любом месте вы хотите, это причина.

$(document).ready(function() { 
var $all = $("#all"); 
$all.on("click", function(){ 
    alert("click"); 
}); 
}); 

Answer 2

У вас есть ошибка в логике обработчика кликов; Вам просто нужно пройти анонимную функцию обработчика:

$all.on("click", function(){ 
    tabClicked("all"); 
}); 

Ваш оригинальный $(function() {}); эквивалентно $(document).ready(), следовательно, он был немедленно выполнен, независимо от того, была ли нажата элемент.

Answer 3

заменить $ (функция() для функции()