ToggleButton: onChange arguments could be more useful

[Colin Snover]

Currently, only the boolean value of the checked attribute is provided when the onChange event is fired, which is worthless in situations where the same event handler is bound to multiple different checkboxes/radiobuttons/whatever, which I imagine is a fairly common case. I’m not sure what the best option here would be; either adding value as a second argument or provide the element itself as an argument or something else.

[bill]

The widget itself is already available as "this":

function myOnChangeHandler(checked){
    console.log(this.id + ": value= " + checked);
}

[Colin Snover]

dojo.declare('foo', [ dijit._Widget, dijit._Templated ], {

    templateString: '<div><input dojoType="dijit.form.Checkbox" dojoAttachEvent="onChange:_foo"></div>',

    _foo: function () {
        console.log(this); // foo
    }
});

function myOnChangeHandler(checked){
    console.log(this); // window
}
dojo.connect(checkbox, 'onChange', window, 'myOnChangeHandler');

[bill]

The dojo.connect() call in your code example should be:

dojo.connect(checkbox, 'onChange',  myOnChangeHandler);