Function working except when you click it first time, Chrome

Question

I have created a JavaScript onclick function. It works fine in Firefox. but in Chrome/IE, function working except when you click it first time. My code is below:





Untitled Document

Arun P Johny · Accepted Answer

The keepCount() method is used to register a on change event handler, so you need to execute it on document load, not on click of the input element.

So what happens is when the first click happens there is no onchange event handler registered, so it will not get fired, and it registers a onchange event then. When the second click happens you have a onchange event handler registered so it gets fired
The problem with this is as you keep executing KeepCount method the every click will make the onchange habdler to fire multiple times

Ex:

function KeepCount() {

    var mytextbox = document.getElementById('onclickval1');
    var myform = document.getElementById('compare-meet');
    var inputElems = myform.getElementsByTagName("input");
    for (var i=0; i 5)
                {
                    alert('Pick Just Four Please!')
                    this.checked = false
                    mytextbox.value = mytextbox.value.replace(this.value+",", "")
                    document.comparepage; return false;
                }
                if (n < 3)
                {
                    document.getElementById('compsend').disabled=true;
                }
                if (n > 2)
                {
                    document.getElementById('compsend').disabled=false;
                }
            }else{
                mytextbox.value = mytextbox.value.replace(this.value+",", "");
                s = document.getElementById("onclickval1").value;
                n = s.split(',').length; 
                if (n < 3)
                {
                    document.getElementById('compsend').disabled=true;
                }
                if (n > 2)
                {
                    document.getElementById('compsend').disabled=false;
                }
            }
        }
    }
}

window.onload = KeepCount;

And remove the onclick="KeepCount()" from the checkboxes

Demo: Fiddle

Function working except when you click it first time, Chrome

Answers (2)

Related Questions