CODEWITHSUNDEEP
javascriptc#asp.net-mvcasp.net-mvc-helpers
Jimmy Lillegaard
Jimmy Lillegaard

Reputation: 129

Change an TextBoxFor readonly property on whether a checkbox is checked or not

I'm trying to change an TextBoxFor readonly property on whether a checkbox is checked or not. I have read other posts but the answers don't seem to work for me.. I any one please would point out what i miss in the code below.

<div class="editor-label">
                @Html.LabelFor(m => m.AddFitbit)
            </div>
            <div class="editor-field" id="checkBox">
                @Html.CheckBoxFor(m => m.AddFitbit)
            </div>
            <div class="editor-label">
                @Html.LabelFor(m => m.Mail)
            </div>
            <div class="editor-field" id="userMail">
                @Html.TextBoxFor(m => m.Mail)
            </div>
            <br />
            <input class="submit" type="submit" value="@LangResources.Create" />
            <button onclick="history.back(); return false;">@LangResources.BtnCancel</button>
        </div>
    </article>
}

@Html.ValidationSummary(true, LangResources.ScreenAddGeneralError)

<script type="text/javascript">
    $(function() {
        $('#checkBox').change(function () {
            if ($(this).is(':checked')) {
                $('#userMail').attr('readonly');
            } else {
                $('#userMail').removeAttr('readonly');
            }
        });
    });
</script>

Upvotes: 0

Views: 576

Answers (1)

Zolt&#225;n Tam&#225;si
Zolt&#225;n Tam&#225;si

Reputation: 12764

You target the div which contains your checkbox, which indeed doesn't trigger any change events. Target the checkbox inside the div like this.

$(function() {
    $('#checkBox input[type="checkbox"]').change(function () {
        $('#userMail input[type="text"]').prop('readonly', $(this).is(':checked'));
    });
});

Also, I corrected and simplified your code by using prop instead of attr. Attributes should mean an initial value of a given property, so it's a better practice to change the corresponding element property instead.

For details about using prop, see the documentation (based on @Liam's comment).

Upvotes: 3

Related Questions