Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#13942 closed defect (fixed)

dojox.mobile.CheckBox "value" attribute not set

Reported by: Philippe May Owned by: ykami
Priority: high Milestone: 1.7
Component: DojoX Mobile Version: 1.7.0b1
Keywords: 1.7-mobile Cc: Douglas Hays, Atsushi Ono
Blocked By: Blocking:

Description

For example: in dojox/mobile/tests/test_FormControls.html: dijit.byId('cbox').get('value') returns always "on".

Attachments (1)

CheckBox.js.diff​ (347 bytes) - added by Philippe May 10 years ago.
Proposed patch

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by Philippe May

Attachment: CheckBox.js.diff​ added

Proposed patch

comment:1 Changed 10 years ago by ykami

Cc: Douglas Hays Atsushi Ono added
Keywords: 1.7-mobile added
Status: new → assigned

comment:2 Changed 10 years ago by ykami

Resolution: → fixed
Status: assigned → closed

In [26605]:

Fixes #13942 !strict Added _getValueAttr, which was taken from dijit/form/CheckBox.js. Thanks onoat for your investigation.

comment:3 Changed 10 years ago by Atsushi Ono

Thanks for reporting this and proposing a patch, aprilmay. dijit.form.CheckBox already has _getValueAttr function, which returns the checkbox value ("on" if the value is not specified) if checked, and false if not checked, so the fix above added the same function to dojox.mobile.CheckBox.

Note: See TracTickets for help on using tickets.