Javascript toggle checkbox

Question

I need to toggle all buttons with a single function. Function needs to toggle all checkboxes in the document as my checkboxes are freestanding and not part of a form.

I currently have this, but it is not working properly. I get syntax error: syntax error in my firefox console.

    checked=false;
    function checkedAll() {
        var c = new Array();
        c = doc.getElementsByTagName('input');
        if (checked == false){
            checked = true;
        }else{
            checked = false;
        }
        for (var i = 0; i < c.length; i++){
            if (c[i].type == 'checkbox'){
                c[i].checked = checked;
            }
        }
    }

How can I fix my code?

Thanks

possible duplicate of How do I check a checkbox with jQuery or JavaScript? — I wrestled a bear once.
– I wrestled a bear once., Commented Oct 3, 2013 at 9:24
FYI, your if statement can be replaced with just checked = !checked;. The ! character negates your checked variable. If checked was equal to true, !checked would equal false. — James Donnelly
– James Donnelly, Commented Oct 3, 2013 at 9:25

Ali Seyfollahi · Accepted Answer · 2018-03-17 09:29:41Z

3

Two main items to refactor. First, instead of doc it must be document. Second instead of relying on a global just pass in a boolean to determine whether or not to check the checkboxes.

function checkedAll(isChecked) {
    var c = document.querySelectorAll('input[type="checkbox"]');

    for (var i = 0; i < c.length; i++){
        c[i].checked = isChecked;
    }
}

JS Fiddle: http://jsfiddle.net/Jvnfm/107/

edited Mar 17, 2018 at 9:29

Ali Seyfollahi

2,9602 gold badges24 silver badges29 bronze badges

answered Oct 3, 2013 at 9:25

Kevin Bowersox

94.6k19 gold badges164 silver badges198 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

user9027325 · Accepted Answer · 2018-07-05 22:07:03Z

2

You can alternatively perform the following for each checkbox element:

    c[i].click();

This version will trigger any associated event handlers associated with that element.

answered Jul 5, 2018 at 22:07

user9027325

1452 silver badges6 bronze badges

1 Comment

oxwilder Over a year ago

bad idea if the script is being run by an onclick event

Collectives™ on Stack Overflow

Javascript toggle checkbox

2 Answers 2

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related