Как сделать запрет на повторный клик JS?

Question

[DeniSidorenko]

Привет, есть обычное меню которая открывается при клике на див и закрывается при клике на любое место веб документа, кроме при клике на его дочерние элементы и на него самого. ( скрипт ниже). Так же случается баг что я открываю его и нажимаю еще раз на кнопку открытия происходит конфликт ибо , при клике на ту кнопку он исчезает ибо клик за пределы его но потом снова появляется ибо та кнопка отвечает за его вызов.

Вызов

$('.click').on('click',function({
$('.block').show();
})

Скрываем

$(document).mouseup(function (e){ // событие клика по веб-документу
  var div = $(".block"); // тут указываем ID элемента
  if (div.data('loading')) return; // если загружается, ничего не делаем

  if (!div.is(e.target) // если клик был не по нашему блоку
      && div.has(e.target).length === 0) { // и не по его дочерним элементам
    div.hide("slide", {direction: "left"} , 1000);
  }
});

Comments

[napa3um]

https://learn.javascript.ru/event-bubbling

Answer 1

[Алексей]

А что по вашему должно происходить при повторном клике? Меню должно свернутся и больше не разворачиваться или вообще не возможно было бы нажать повторно?
Как правило на кнопку меню вешают не метод show, а метод trigger, который если элемент скрыт, то он его показывает, если он отображается, то его скрывают.

Comments

[Дмитрий Макаров]

Toggle

[Алексей]

Дмитрий Макаров: да, верно, toggle()

[kulaeff]

Согласен и могу дополнить обоснованием: когда юзер открывает меню (или любую другую всплывающую хрень), то возникает, так сказать, новый контекст для юзера, новый уровень, слой или хз как это называть. Поэтому клик вне всплывающей хрени, даже если клик был по кнопке, которая открывает эту хрень, должен закрывать хрень обратно. Это логично и, думаю, подобного поведения ожидает большинство пользователей. Так что я бы не стал заморачиваться и оставил как есть.