how to change textbox background color through javascript?

Question

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

backgroungColor should be backgroundColor, but maybe just a typo? — Felix Kling
– Felix Kling, Commented Aug 5, 2010 at 12:27
it backgroundColor in actual program I just missed it here only — nectar
– nectar, Commented Aug 5, 2010 at 12:32
What about in function color(traget)? Is that just a typo here as well? — Danny Nimmo
– Danny Nimmo, Commented Aug 5, 2010 at 12:33

Maciej Kucharz · Accepted Answer · 2010-08-05 12:31:51Z

6

With jQuery you could try this:

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

answered Aug 5, 2010 at 12:31

Maciej Kucharz

1,4431 gold badge14 silver badges17 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

nectar · Accepted Answer · 2010-08-05 13:00:30Z

3

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

answered Aug 5, 2010 at 13:00

nectar

9,69736 gold badges82 silver badges101 bronze badges

1 Comment

user2672165 Over a year ago

Seems to change border color. Not background color as asked for.

Álvaro González · Accepted Answer · 2010-08-05 12:57:15Z

2

Typo?

backgroungColor
         ^

Update

Typo?

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

Seriously, actual code does matter.

edited Aug 5, 2010 at 12:57

answered Aug 5, 2010 at 12:28

Álvaro González

147k45 gold badges282 silver badges378 bronze badges

1 Comment

nectar Over a year ago

its backgroundColor in actual program I just missed it here only

Pyae Phyoe Shein · Accepted Answer · 2010-08-05 12:59:28Z

2

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

function color(traget)

answered Aug 5, 2010 at 12:59

Pyae Phyoe Shein

13.9k47 gold badges153 silver badges244 bronze badges

Comments

Danny Nimmo · Accepted Answer · 2010-08-05 12:27:57Z

1

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

answered Aug 5, 2010 at 12:27

Danny Nimmo

6947 silver badges22 bronze badges

1 Comment

nectar Over a year ago

its backgroundColor in actual program I just missed it here only

user395702 · Accepted Answer · 2010-08-05 12:30:58Z

1

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

and write it as color(uid);

answered Aug 5, 2010 at 12:30

user395702

1066 bronze badges

1 Comment

Danny Nimmo Over a year ago

It's supposed to be a string there. <edit>S</edit>He's using getElementById() to retrieve the element.

Collectives™ on Stack Overflow

how to change textbox background color through javascript?

6 Answers 6

Comments

1 Comment

Update

1 Comment

Comments

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

1 Comment

Update

1 Comment

Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related