Как сделать разумнее - событие на click

[Zverushka]

Суть в том, что у меня внутри плагина модального окна jquery есть 2 функции. Одна скрывает модальное окно, другая показывает

var modalHide = function() {
$modal.fadeOut(delayHide);
$modal.toggleClass('active');
};
var modalShow = function(event) {
// on bodyclick hide modal
if (bodyClick) {
$("body").click(function(){	
if ($modal.hasClass("active")) {
	 modalHide();
}
});
$context.click(function(event){
event.stopPropagation();
});
}
$modal.fadeIn(delayShow);
setTimeout(function(){
$modal.toggleClass('active');
}, delayShow);
}

Суть в том, что при показе модального окна и вызове функции modalShow() - я вешаю обработчик события клика по body - чтобы окно исчезало, если кликнули не по нему.

Стоит ли в функции modalShow - привязывать событие на клик по боди, а в функции modalHide - его "отвязывать"? делать unbing. Или оставить как есть? Как было бы "правильнее", оптимизированнее, логичнее и "красивее"?

Edited October 27, 2013 by Zverushka

[wwt]

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

И кстати вы используете setTimeout а в jquery есть такой метод delay()

$modal.fadeIn(delayShow).delay(delayShow).toggleClass('active');

[Zverushka]

Пробовала delay - не срабатывает. Зато мне подсказали в другой теме - просто в конце fadein функцию вставить Уже изменила скрипт.

[wwt]

Пробовала delay - не срабатывает. Зато мне подсказали в другой теме - просто в конце fadein функцию вставить Уже изменила скрипт.

А ну да. delay() устанавливает задержку для эффектов )) моя ошибка.