Question

Hi i have a textbox field which can have a number between 1 – 6 or more.
1-3 is green
4-5 is orange
6 or more is red

this field auto calculates based on value selected from other fields.

function changeColour() {
if (document.getElementById("Q49I1029").value > 0) {  
document.getElementById("Q49I1029").style.background = '#00FF33'; // Green
}
else if (document.getElementById("Q49I1029").value > 3) {
document.getElementById("Q49I1029").style.background = '#EC582D'; // Orange
}
else (document.getElementById("Q49I1029").value > 5) {
document.getElementById("Q49I1029").style.background = '#990000'; // Reg
}
alert ("This box must be filled!");
document.getElementById("Q49I1029").focus();
return false;
}

but i am unable to get this to work.
any help will be much appriciated.
thanks

Answer 1

Use IF statement only for all condition.

Remove “else if” and “else”.