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

Question

[EVGEK]

$(function(){
  $('.delivery_list').click(function(){
    $(".cities_list").slideToggle('fast');  // показываем список при клике
  });

  $('ul.cities_list li').click(function(){
    var tx = $(this).html();
    $(".cities_list").slideUp('fast');
    $(".delivery_list span").html(tx); // заменяет выбранный пункт меню
  }); 
  $(document).click(function(e){ 
    if ($(e.target).closest(".cities_list").length);    // закрываем список если кликаем мимо него
    $('.cities_list').hide(); 
    e.stopPropagation(); 
  }); 
})



//Срабатывает только при двойном клике . по .delivery_list' . при одиночном не работает . хотя без

$(document).click(function(e){ 
  if ($(e.target).closest(".cities_list").length);    // закрываем список если кликаем мимо него
  $('.cities_list').hide(); 
  e.stopPropagation(); 
}
//работает при одиночном .

подскажите люди добрые.

Answer 1

[Stalker_RED]

А что за странные конструкции
if ( ... );

Эти if-ы ничего не делают же.

Может стоит переписать как-то так

if ( условие ) {
   // действие которое сработает только при выполнении условия
}

Comments

[EVGEK]

Спасибо, добавил как вы посоветовали, работает при одиночном, но не закрывает при нажатии на любую область документа . может что то не так с $(document).click(function(){ ?? не пойму .

[Stalker_RED]

EVGEK: А как вы определяете что клик был именно мимо? Я могу только догадываться что такое ".cities_list"

[EVGEK]

Спасибо . слои накладывались друг на друга . поэтому и закрывает и открывает одновременно при событии . разъединил слои . теперь все ок. благодарю за правку и быстрый ответ.