Unable to disable a textbox using JavaScript

Question

Depending on which option I choose in the above drop down box, it is supposed to either gray the text boxes out, or let the user type in them. I am using the following:

document.getElementById(\'freq2\').disabled=true;

This does not work. But, I can use the following just fine, and it successfully disappears:

document.getElementById(\'freq2\').style.display = \'none\';

Why can I not gray out the text boxes, but I can hide them, and change their colors, etc.?

Sorry, that it was I have. It is not working. Will edit above. Any other tips? — user2537383
– user2537383, Commented Jul 11, 2013 at 19:14
It's working here: jsfiddle.net/6A8Zd Why are you escaping the quotes? Are these statements part of another string? — colestrode
– colestrode, Commented Jul 11, 2013 at 19:20
So have you confirmed that the quotes are output in the rendered source correctly? — Scott
– Scott, Commented Jul 11, 2013 at 19:23

Scott · Accepted Answer · 2013-07-11 19:40:24Z

1

You need to unescape your quotes.

document.getElementById(\'freq2\').disabled=true;

needs to be

document.getElementById('freq2').disabled=true;

or the JavaScript won't be valid, so it won't disable it.

You can wrap multiple textboxes in a fieldset

<fieldset id="myfields" disabled="disabled">
    <input type="text" />
    <input type="text" />
</fieldset>

JavaScript:

document.getElementById("myfields").disabled = true;

CSS to remove border:

fieldset {
    border: 0;
}

JSFiddle

edited Jul 11, 2013 at 19:40

answered Jul 11, 2013 at 19:22

Scott

21.6k8 gold badges67 silver badges73 bronze badges

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

4 Comments

user2537383 Over a year ago

I can mass disable them, thanks. It did create some ugly border around it, messing with my format, though. Any tips?

Scott Over a year ago

@user2537383 You can hide it using the CSS I have added in my question. If it has helped you please upvote, and mark as answer. Thanks

user2537383 Over a year ago

I definitely will mark yours. Thanks for all the help. Just one last thing (hopefully). How can I have the boxes enable by default rather than disabled?

Scott Over a year ago

You're welcome. Just remove disabled="disabled" from the fieldset.

colestrode · Accepted Answer · 2013-07-11 19:15:06Z

1

You're looking for the disabled property (not disable). Also, no need to escape your quotes

document.getElementById('freq2').disable=true;

Working Demo

answered Jul 11, 2013 at 19:15

colestrode

10.7k3 gold badges32 silver badges43 bronze badges

1 Comment

user2537383 Over a year ago

Sorry, that it was I have. It is not working. Will edit above. Any other tips?

javitube · Accepted Answer · 2013-07-11 19:20:10Z

0

Try one of theese:

document.getElementById(\'freq2\').setAttribute('disabled', 'disabled');

Or

document.getElementById(\'freq2\').setAttribute('disabled', true);

Sample: http://jsfiddle.net/javitube/wzzHC/

answered Jul 11, 2013 at 19:20

javitube

1011 silver badge2 bronze badges

Collectives™ on Stack Overflow

Unable to disable a textbox using JavaScript

3 Answers 3

4 Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

4 Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related