Почему не работает массив ссылок jquery?

Question

[idclev31]

На странице есть определенное количество элементов-ссылок внутри<div class='roomsList'> ,структура у них такая <a href=""><span></span></a>
Так же имеется javascript код,который в идеале должен при нажатии на ссылку,выводить сообщение в консоль

$('.roomsList a span').toArray().forEach(function(element){
    $(element).on('click',function(){
      console.log('lalala');
      return false;
    });
  });

По идее return false должен предотвращать перезагрузку страницы,но она все равно происходит.Что здесь я делаю не так?
Я совсем забыл,что для динамически создаваемых обьектов не вешается слушатель,поэтому-то ничего и не работает.
Проблема решается делегированием,и вешанием слушателя на div

Answer 1

[Kir ---]

Зачем тут toArray?

$('.roomsList a').each(function(element){
   //...
});

А вообще вам нужно при клике делать preventDefault

$(this).on('click',function(event){
    event.preventDefault();
});

Comments

[idclev31]

Возможно и незачем,но к сожалению все равно перезагружается

[Kir ---]

idclev31: F5 нажмите

[idclev31]

Так,я извиняюсь,вы правы,безусловно,у меня что-то с ssh,почему-то файл не передается,а я и не заметил,спасибо вам!

[idclev31]

Хотя теперь я все поправил,но все равно происходит перезагрузка.

Answer 2

[Nikolay Talanov]

Вы отменяете стандартное действие по клику на span (которого в общем то и нет), и затем идет всплытие ивента, которое триггерит стандартный ивент при клике на анкор. Вам либо надо предотвращать всплытие, с помощью e.stopPropagation(), либо биндить перехват ивента конкретно на a, а не на спан внутри a.
Ну и к тому же конструкция с toArray.forEach кривая, вам надо делать как написано в ответе Kir ---