2017-05-30 59 views
1

现在,我有以下几点:两个div切换,多个JavaScript点击挂钩合并成一个

<script> 
    $('#nav-close').on('click', function(){ 

     var mover = $('#mover'); 

     // register is not visible 
     if(mover.css('opacity') === '1'){ 
     mover.css({'opacity': '0'}); 
     } else { 
     mover.css({'opacity': '1'}); 
     } 

    }); 
    $('#nav-expander').on('click', function(){ 

     var mover = $('#mover'); 

     // register is not visible 
     if(mover.css('opacity') === '1'){ 
     mover.css({'opacity': '0'}); 
     } else { 
     mover.css({'opacity': '1'}); 
     } 

    }); 
</script> 

如何结合这两种成一个 - 逻辑,如下面的更有效的脚本:

$('#nav-close' || '#nav-expander').on('click', function(){ 

...

+3

替换||用逗号? – RST

+0

试过 - 没有运气......('#nav-close','#nav-expander')... – Davo

回答

2

做与,代替|| .Refer jquery multiple selector documentation

$('#nav-close,#nav-expander').on('click', function() { 
    var mover = $('#mover'); 
    // register is not visible 
    if (mover.css('opacity') === '1') { 
    mover.css({ 
     'opacity': '0' 
    }); 
    } else { 
    mover.css({ 
     'opacity': '1' 
    }); 
    } 

}); 
+1

工作过!大! – Davo

0
Please see:- 
    Multiple Selector at => http://api.jquery.com/multiple-selector/ 
    Note, the double quotes around your div selectors. ;) 

$("#nav-close, #nav-expander").click('click', function() { 
var mover = $('#mover'); 
// register is not visible 
if (mover.css('opacity') === '1') { 
    mover.css({ 
    'opacity': '0' 
}); 
} else { 
    mover.css({ 
    'opacity': '1' 
}); 
} 

});