2013-07-09 3 views
-1

현재 코드가 너무 복잡합니다. 더 효율적으로 만들 수 있도록 도와주세요.탐색 버튼 클릭 스타일 변경 효율성

function buttons(){ 
    $('.notclicked').click(function(){ 
     $('.leftmenu').addClass('leftclicked'); 
     $('.mainwrap').addClass('mainclicked'); 
     $(this).removeClass('notclicked'); 
     $(this).addClass('clicked'); 
     $('.clicked').click(function(){ 
      $('.leftmenu').removeClass('leftclicked'); 
      $('.mainwrap').removeClass('mainclicked'); 
      $(this).removeClass('clicked'); 
      $(this).addClass('notclicked'); 
      buttons(); 
     }); 
     buttons(); 
    }); 
}; 
buttons(); 

(http://quinnkeaveney.com/recondite/brian/index.php)

+0

코드에 문제가 없다면 http://codereview.stackexchange.com/에 더 적합합니다. – Dom

답변

2

아직로드되지 않았을 수 있습니다 선택기에 클릭을 결합하는

$(document).on('click', selector, function() { 
    // your function 
} 

를 사용할 수 있습니다.

0
$('#menu').click(function(){ 
    var btn = $(this); 
    // find out if it has the notclicked class 
    var notClicked = btn.hasClass('notclicked'); 
    // when notClicked is true, toggleClass acts like addClass 
    // when notClicked is false, toggleClass acts like removeClass 
    $('.leftmenu').toggleClass('leftclicked', notClicked); 
    $('.mainwrap').toggleClass('mainclicked', notClicked); 
    // This will toggle these both back and forth 
    btn.toggleClass('notclicked clicked'); 
}); 
관련 문제