CODEWITHSUNDEEP
javascriptjqueryhtml
Ajay Gopal Shrestha
Ajay Gopal Shrestha

Reputation: 199

CheckAll/UncheckAll checkbox with jQuery

I have made a check-box checkall/uncheckall.

HTML

<div> Using Check all function </div>
<div id="selectCheckBox">
<input type="checkbox" class="all" onchange="checkAll('selectCheckBox','all','check','true');" />Select All
<input type="checkbox" class="check" onchange="checkAll('selectCheckBox','all','check','false');" />Check Box 1
<input type="checkbox" class="check" onchange="checkAll('selectCheckBox','all','check','false');" />Check Box 2
<input type="checkbox" class="check" onchange="checkAll('selectCheckBox','all','check','false');" />Check Box 3
<input type="checkbox" class="check" onchange="checkAll('selectCheckBox','all','check','false');" />Check Box 4
</div>

main.js

function checkAll(parentId,allClass,checkboxClass,allChecked){
    checkboxAll = $('#'+parentId+' .'+allClass);
    otherCheckBox = $('#'+parentId+' .'+checkboxClass);
    checkedCheckBox = otherCheckBox.filter($('input[type=checkbox]:checked'));
    if(allChecked=='false'){
        if(otherCheckBox.size()==checkedCheckBox.size()){
            checkboxAll.attr('checked',true);
        }else{
            checkboxAll.attr('checked',false);
        }
    }else{
        if(checkboxAll.attr('checked')){
            otherCheckBox.attr('checked',true);
        }else{
            otherCheckBox.attr('checked',false);
        }
    }
}

It works fine. But get bulky when I have whole lot of checkboxes. I want to do same work by using jQuery rather than putting onchange on each checkbox. I tried different sort of things but couldnot work. I tried following one:

$('.check input[type="checkbox"]').change(function(e){
    checkAll('selectCheckBox','all','check','true');
});

to do same work as onchange event but didnot work. Where do I went wrong.

Upvotes: 3

Views: 1731

Answers (3)

PSL
PSL

Reputation: 123739

I think you just need this: You do not need to pass all the arguments and have the inline onchange event attached to it. You can simplify your code.

$(function () {
    $('input[type="checkbox"]').change(function (e) {
       if(this.className == 'all')
       {
           $('.check').prop('checked', this.checked); //Toggle all checkboxes based on `.all` check box check status
       }
        else
        {
            $('.all').prop('checked', $('.check:checked').length == $('.check').length); // toggle all check box based on whether all others are checked or not.
        }
    });
});

Demo

Upvotes: 2

Xeon
Xeon

Reputation: 5989

Your selector is wrong:

.check input[type="checkbox"]

Above selects any input of type checkbox that has the ancestor with class .check. It'll match this:

<div class="check">
    <input type="checkbox".../> 
</div>

it should be:

input.check[type="checkbox"]

Upvotes: 1

Misa Lazovic
Misa Lazovic

Reputation: 2823

You closed the string here $('.check input[type='checkbox']') instead, you should use double quotes $('.check input[type="checkbox"]')

Upvotes: 0

Related Questions