ansaurus

Question

Answer 1

+2 A:

First, the valid syntax is

jQuery("input[name=ticketID]")

second, have you tried:

jQuery(":radio")

instead?

third, why not assign a class to all the radio buttons, and select them by class?

mkoryak 2009-09-10 17:13:27

Actually "input[name='ticketID']" is valid syntax, the single quotes are just not necessary.

karim79 2009-09-10 17:21:52

why not just use the name? and select them by that? it should work fine; i've done it a million times.

geowa4 2009-09-10 17:47:09

I agree with mkoryak: "assign a class to all the radio buttons, and select them by class"

Floetic 2009-09-10 18:25:25

then you have to maintain both classnames and names when names suffice

geowa4 2009-09-10 18:59:51

jQuery selector speeds - classes are faster than attributes

gnarf 2009-09-10 19:37:33

i think we should solve his problem before making optimizations.

geowa4 2009-09-10 20:12:48

Answer 2

A:

You should use classes to make it more simple, instead of the attribute name, or any other jQuery selector.

rogeriopvl 2009-09-10 17:13:55

let me guess, you use rails?

geowa4 2009-09-10 17:17:06

Actually I don't. Can you elaborate?

rogeriopvl 2009-09-10 17:19:53

rails has that array notation stuff for checkboxes. but you can have multiple two items with the same name without "[]"

geowa4 2009-09-10 17:21:58

Yes you're right... I forgot completely about that. Going to edit the post.

rogeriopvl 2009-09-10 17:25:21

Answer 3

+3 A:

I've refactored your code a bit, this should work:

jQuery("#loadActive").click(writeData);

function writeData() {
    jQuery("#chatTickets input:radio").attr('disabled',true);
}

If there are more than two radio buttons on your form, you'll have to modify the selector, for example, you can use the starts with attribute filter to pick out the radios whose ID starts with ticketID:

function writeData() {
    jQuery("#chatTickets input[id^=ticketID]:radio").attr('disabled',true);
}

karim79 2009-09-10 17:28:13

that'll disable every radio button in that form. in his question there might only be two, but there are likely going to be more.

geowa4 2009-09-10 17:45:44

@geowa4 - true, I assumed those were the only radios on his form. In any case, I've edited to be a bit more useful.

karim79 2009-09-10 17:54:55

+1 for removing the useless `.each()`; `.attr()` will apply to all selected elements anyway.

gnarf 2009-09-10 19:40:03

Answer 4

+1 A:

I just built a sandbox environment with your code and it worked for me. Here is what I used:

<html>
  <head>
    <title>test</title>
    <script src="http://ajax.googleapis.com/ajax/libs/jquery/1.3.2/jquery.min.js"&gt;&lt;/script&gt;
  </head>
  <body>
    <form id="chatTickets" method="post" action="/admin/index.cfm/">
      <input id="ticketID1" type="radio" checked="checked" value="myvalue1" name="ticketID"/>
      <input id="ticketID2" type="radio" checked="checked" value="myvalue2" name="ticketID"/>
    </form>
    <a href="#" title="Load ActiveChat" id="loadActive">Load Active</a>

    <script>
      jQuery("#loadActive").click(function() {
        //I have other code in here that runs before this function call
        writeData();
      });
      function writeData() {
        jQuery("input[name='ticketID']").each(function(i) {
            jQuery(this).attr('disabled', 'disabled');
        });
      }
    </script>
  </body>
</html>

I tested in FF3.5, moving to IE8 now. And it works fine in IE8 too. What browser are you using?

geowa4 2009-09-10 17:39:34

Answer 5

A:

code:

function writeData() {
    jQuery("#chatTickets input:radio[id^=ticketID]:first").attr('disabled', true);
    return false;
}

See also: Selector/radio, Selector/attributeStartsWith, Selector/first

ranonE 2009-09-10 18:22:37

ansaurus

tags:

views:

answers:

Disabling radio buttons with jQuery

related questions