Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#10331 closed defect (fixed)

ToggleButton needs the optional priorityChange boolean so attr('checked',value) can be called w/o firing onChange

Reported by: Douglas Hays Owned by: Douglas Hays
Priority: high Milestone: 1.5
Component: Dijit - Form Version: 1.4.0b
Keywords: Cc: Jared Jurkiewicz
Blocked By: Blocking:

Description

_setCheckedAttr always calls _handleOnChange(true) but should allow the user to specify an optional boolean like _setValueAttr does on other form widgets.

Attachments (1)

10331.patch (686 bytes) - added by Douglas Hays 9 years ago.
_setCheckedAttr now takes an optional priorityChange boolean parameter

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by Douglas Hays

Attachment: 10331.patch added

_setCheckedAttr now takes an optional priorityChange boolean parameter

comment:1 Changed 9 years ago by Douglas Hays

Cc: Jared Jurkiewicz added

comment:2 Changed 9 years ago by Douglas Hays

Resolution: fixed
Status: newclosed

(In [21031]) Fixes #10331. Add optional priorityChange boolean to _setCheckedAttr to conditionally avoid onChange. Add automated CheckBox? tests.

comment:3 Changed 9 years ago by bill

(In [22388]) Get submit tests running on IE8. On IE8 (at least on my machine) we need a delay between form submits. Refs #10331 ([21031]).

comment:4 Changed 8 years ago by bill

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