Help, please, make this code dynamic, it’s just for one box check only.

function logincheckboxCheck() { if (document.getElementById("checkbox1").checked) { document.getElementById("checkboxDiv").className = "checkboxOn"; } else { document.getElementById("checkboxDiv").className = "checkboxOff"; } } 

    2 answers 2

     function logincheckboxCheck(el){ if($(el).is(":checked")){ $("#checkDiv").addClass("checkboxOn"); }else{ $("#checkDiv").addClass("checkboxOff"); } } 

    Stupidly pass an element to a function or:

     <input type="checkbox" onClick="logincheckboxCheck(this)"> 
       function logincheckboxCheck(el) { if (el.checked) { el.className = "checkboxOn"; } else { el.className = "checkboxOff"; } } 

      It is better to use another event, because you can not click on the input directly, but also on the label, which will make the checkbox marked

       <input type="checkbox" onChange="logincheckboxCheck(this)">