jquery if statement is not working correctly

Question

I am having problem with my jquery code, I am trying to state that if "Yes" button is on, disable the "No" button, else if the "No" button is on, then disable the "Yes" button, else enable both buttons.

But at the moment do button is being disabled when the other button is turned on, does anyone know why this is?

Below is code:

function btnclick(btn)
{
    var context = $(btn).parents('#optionAndAnswer');
    if (context.length == 0) {
        context = $(btn).parents('tr');
    }

$(btn).toggleClass("answerBtnsOff");
$(btn).toggleClass("answerBtnsOn");

    if ($("#answerYes", context).hasClass('.answerBtnsOn')) {
        $("#answerNo", context).attr("disabled", "disabled");
    }

    else if ($("#answerNo", context).hasClass('.answerBtnsOn')) {
        $("#answerYes", context).attr("disabled", "disabled");
    }

    else {
    $("#answerYes", context).removeAttr("disabled");
    $("#answerNo", context).removeAttr("disabled");
    }


    return false;
}

below is html for both yes and no buttons:

Yes button:

<input class="answerBtns answers answerBtnsOff" name="answerYesName"   id="answerYes"   type="button"   value="Yes"     onclick="btnclick(this);"/>

No Button:

<input class="answerBtns answers answerBtnsOff" name="answerNoName"    id="answerNo"        type="button"   value="No"      onclick="btnclick(this);"/>

How is this question any different than your other questions recently on StackOverflow — Metro Smurf
– Metro Smurf, Commented Jul 11, 2012 at 0:14
Oh I did not know about these questions, there is a group of us and we use this account to ask any programming questions we may have, some of our group members must of asked some questions related to this application, but this question is asking how to disable and undisable buttons — user1490145
– user1490145, Commented Jul 11, 2012 at 0:19
Isn't this type of thing what radio buttons are for? If you want to use push buttons wouldn't it be better to make it that clicking "Yes" automatically deselects "No" rather than disabling it? Otherwise how does the user change their mind? — nnnnnn
– nnnnnn, Commented Jul 11, 2012 at 0:20

Musa · Accepted Answer · 2012-07-11 00:10:35Z

2

You dont put the . in toggleClass/hasClass/removeClass/... you only use them in selectors

answered Jul 11, 2012 at 0:10

Musa

97.9k17 gold badges123 silver badges144 bronze badges

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

Comments

JoeCortopassi · Accepted Answer · 2012-07-11 00:11:38Z

0

Change this .attr("disabled", "disabled"); to this .attr("disabled", true); for both buttons. Plus, you will want to remove the period like @Musa said

answered Jul 11, 2012 at 0:11

JoeCortopassi

5,0939 gold badges39 silver badges65 bronze badges

1 Comment

codeandcloud Over a year ago

are you mentioning .prop("disabled", true);? stackoverflow.com/a/1414366/17447

Collectives™ on Stack Overflow

jquery if statement is not working correctly

2 Answers 2

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related