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);