jQuery select checkbox

Question

I am using jquery to select checkbox on and off when user click on image.

Here is my code:

function clickimage(e){
    chkbox = e.parent().children('.chkBox');
    if(chkbox.is(":checked")){
        chkbox.attr('checked', false);
    }else{
        chkbox.attr('checked', true);
    }
}

When I first click on image, the checkbox does checked. When I click on image again, the checkbox is unchecked. But the problem is after that, the checkbox will never be selected again.

I try to verify chkbox.is(":checked") and it does return false value. I put the checking code after chkbox.attr('checked', true);, it also return false value.

Why is it like that? How can I fix it?

Thank you.

can you share the html

Milind Anantwar
– Milind Anantwar

2014-04-13 13:36:48 +00:00
Commented Apr 13, 2014 at 13:36 — Milind Anantwar
– Milind Anantwar, Commented Apr 13, 2014 at 13:36

Amit Joki · Accepted Answer · 2014-04-13 13:38:05Z

2

Do this:

function clickimage(e){
    chkbox = e.parent().children('.chkBox');
    if(chkbox.is(":checked")){
        chkbox.prop('checked', false);
    }else{
        chkbox.prop('checked', true);
    }
}

You should use .prop() and not .attr()

answered Apr 13, 2014 at 13:38

Amit Joki

59.4k7 gold badges80 silver badges96 bronze badges

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

Comments

urbz · Accepted Answer · 2014-04-13 13:48:23Z

1

You do not need Javascript for this.

Just put the image into a label.

Fiddle here

edited Apr 13, 2014 at 13:48

answered Apr 13, 2014 at 13:40

urbz

2,6671 gold badge23 silver badges30 bronze badges

2 Comments

user1995781 Over a year ago

Thanks for your answer. I tried with label. But it doesn't work well on IE8 for my case. That's why I am thinking to use javascript to do the job.

urbz Over a year ago

No problem. I think this question with provided accepted answer is of interest for you then: stackoverflow.com/questions/15740206/…

Fisherman · Accepted Answer · 2014-04-13 13:37:26Z

0

Try

if(chkbox.is(":checked")){
    chkbox.removeAttr('checked');
}else{
    chkbox.attr('checked', 'checked');
}

answered Apr 13, 2014 at 13:37

Fisherman

6,1311 gold badge33 silver badges37 bronze badges

Comments

Milind Anantwar · Accepted Answer · 2014-04-13 13:38:19Z

0

Try this:

 function clickimage(e){
  chkbox = e.parent().find('.chkBox');
  if(chkbox.is(":checked")){
     chkbox.attr('checked', false);
   }else{
      chkbox.attr('checked', true);
   }
 }

answered Apr 13, 2014 at 13:38

Milind Anantwar

82.3k26 gold badges97 silver badges128 bronze badges

Collectives™ on Stack Overflow

jQuery select checkbox

4 Answers 4

Comments

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related