Opened 11 years ago

Closed 11 years ago

Last modified 8 years ago

#7413 closed defect (fixed)

dijit.form.CheckBox: always starts out checked

Reported by: Nathan Toone Owned by: Nathan Toone
Priority: high Milestone: 1.2
Component: Dijit - Form Version: 1.1.1
Keywords: Cc:
Blocked By: Blocking:

Description

When setting our initial values now, we always start out checked. This is due to changes in the attr stuff

Change History (5)

comment:1 Changed 11 years ago by Nathan Toone

Resolution: → fixed
Status: new → closed

(In [14815]) Fixes #7413 - save and restore our initially-checked state !strict

comment:2 Changed 11 years ago by Nathan Toone

(In [14816]) Refs #7413 - we can use params.checked instead of saving to another variable - also fix the reset as well !strict

comment:3 Changed 11 years ago by bill

Resolution: fixed
Status: closed → reopened

Thanks for the fix, it fixes the checked state although actually it's masking the real problem which is that 

this.attr('value', 'foo')

is called during creation which changes the checked state. (There's special handling of 'value' for checkboxes.) I'd like to find a better way to fix this.

comment:4 Changed 11 years ago by bill

Resolution: → fixed
Status: reopened → closed

(In [14823]) Hopefully cleaner fix for CheckBox? initialization vis-a-vis "checked" and "value" parameters.

Fixes #7413, refs #7244 !strict.

comment:5 Changed 8 years ago by bill

Component: Dijit → Dijit - Form
Note: See TracTickets for help on using tickets.