Как правильно написать условие?

Question

[Сергей]

Есть код для закрытия меню по клику на документ:
if (!$(e.target).closest($nav).length) { ... }
Пытаюсь добавить условие закрытия меню при нажатии ESCAPE, но код отрабатывает некорректно:

if (!$(e.target).closest($nav).length || e.keyCode === 27) { ... }

Я так понял, это происходит из-за того, что keyboard-события также имеют свойство target. Им является элемент, который в момент нажатия находится под фокусом, или body, если такого элемента нет.
Какое условие писать в таком случае или единственный вариант - обрабатывать события раздельно?

Вот весь код (меню использует Bootstrap Collapse)

var closeNavbar = function(e) {
    var $nav = $('.navbar-collapse');
    if (!$(e.target).closest($nav).length || e.keyCode === 27) {
      $nav.collapse('hide');
      $(document).off('click keydown', closeNavbar);
    }
  };

  $('#navbar').on('shown.bs.collapse', function() {
    $(document).on('click keydown', closeNavbar);
  });

Comments

[WapSter]

Вы на одно событие пытаетесь повесить 2 e.keyCode не отработается так как оно вешается на документ и называется 'keydown' 'keyup'

[Сергей]

Нажатия срабатывают, но срабатывают любые нажатия, а не клавиши Esc.

[lamer350]

Сергей, что вам мешает вместо вопросов на тостете - повесить e.keyCode в консоль и увидеть что функция получает?

Answer 1

[Robur]

Добавьте проверку по event.type например. Если это мышка то первое условие, если клавиатура, то второе.

Comments

[Сергей]

Кстати, да. Типы событий ведь разные.

Answer 2

[McBernar]

Как вы вешаете на один ивент клавиатуру и мышь?

Comments

[Сергей]

Написал весь код в спойлере. Ивента два, хендлер один.