2013-08-21 2 views
0

Я новичок в этом, но должен быть лучший способ написать это.Как упростить/оптимизировать этот jQuery

Три аналогичные функции, их можно комбинировать с запятой или чем-то еще.

$('#open-sidebar').click(function(){ 
    $("body").toggleClass("menu-active"); 
    return false; 
}); 
$('#close-sidebar').click(function(){ 
    $("body").toggleClass("menu-active"); 
    return false; 
}); 

$("#top").bind("swiperight", function (e) { 
    $("body").toggleClass("menu-active"); 
    return false; 
}); 
$("#content").bind("swiperight", function (e) { 
    $("body").toggleClass("menu-active"); 
    return false; 
}); 
$("#top").bind("swipeleft", function (e) { 
    $("body").toggleClass("menu-active"); 
    return false; 
}); 

$("#content").bind("swipeleft", function (e) { 
    $("body").toggleClass("menu-active"); 
    return false; 
}); 
$("#navigation").bind("swipeleft", function (e) { 
    $("body").toggleClass("menu-active"); 
    return false; 
}); 

Спасибо

+3

Вы должны спросить это на HTTP: // Просмотр Кода. stackexchange.com/. – putvande

ответ

-2
var $body = $(document.body); 

function toggleBody() { 
    $body.toggleClass("menu-active"); 
    return false; 
} 

$('#open-sidebar, #close-sidebar').click(toggleBody); 
$('#top, #content').bind('swiperight swipeleft', toggleBody); 
$('#navigation').bind('swipeleft', toggleBody); 
+1

Ничего себе! Почему нисходящий? – joellustigman

+0

И кто бы ни поддержал этот комментарий, почему они не подняли ответ? O_o – joellustigman

+0

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

0

С этим синтаксисом

$("#navigation, #content, ... ").bind("swipeleft", function (e) { 
    $("body").toggleClass("menu-active"); 
    return false; 
}); 

И вам нужен для мыши также

$('#close-sidebar, #id2, #id3, ....').click(function(){ 
    $("body").toggleClass("menu-active"); 
    return false; 
}); 
+0

AH !, Это то, что я искал. Я делал это неправильно, как это ... $ ("# navigation", "#content", "#top") ... Спасибо всем, я не тестировал остальные – TacoFiesta

+0

@TacoFiesta Теперь посмотрите, задали ли вы свой вопрос с информацией THAT, на ваш вопрос было бы больше по теме. –

Смежные вопросы