my javascript-

function validate_loginform(loginform) 
{
var uid = loginform.uid.value;
var pass = loginform.pass.value;
if(uid == "") 
  {

    color('uid');       
    return false;
  }
if(pass == 0) 
  {
    color('pass');
    return false;
  }

return true;

}

function color(traget)
{
var targetbox = document.getElementById(target);
targetbox.style.backgroundColor="red";
}

but background color is not getting changed even it is not returning fasle开发者_如何学JAVA value. if I remove the color('uid'); nad put alert("user name required"); then this script is working fine.Whats wrong?

it backgroundColor in actual program I just missed it here only

---

With jQuery you could try this:

 $("#textbox").css("background-color", "red");

---

dont call color function, change color inside if condition like-

if(uid == "") 
  {     
    //alert("You must enter User ID.","error");
    loginform.uid.style.borderColor='red';
    loginform.uid.focus();
    return false;
  }

---

Typo?

backgroungColor
         ^

Update

Typo?

function color(traget)
               ^^^^^^
{
var targetbox = document.getElementById(target);

Seriously, actual code does matter.

---

Beware your spelling. It should be "target", not "traget".

function color(traget)

---

You've spelt target wrong in your function header and background wrong in the last line of the function.

---

just remove the single quote (') from color('uid')

and write it as color(uid);