Ext.field.Toggle doesn't fire 'change' event when tapped

[yury.sannikov]

Sencha Touch 2.0.1-RC

Toggle element doesn't raise change event when tapped.

The code below doesn't raise change event in onComponentTap method.

    toggle: function() {
        var value = this.getValue();
        this.setValue((value == 1) ? 0 : 1);


        return this;
    },


    onComponentTap: function() {
        // Toggle the value, and return false so the normal slider functionality doesn't happen
        this.toggle();


        return false;
    }

Actually, change event is raised by Slider's onThumbDragEnd

    onThumbDragEnd: function(thumb, e) {
.....
        this.fireEvent('dragend', this, thumb, this.getValue(), e);
        if (oldValue !== newValue) {
            this.fireEvent('change', this, thumb, newValue, oldValue);
        }
    },

[Jamie Avins]

Thank you for the report.

[yury.sannikov]

Temporary workaround that works for me is:

Ext.define('your.name.space', {
    override: 'Ext.field.Toggle',
    onComponentTap: function() {
        this.callParent(arguments);
        this.fireEvent('change');
        return false;
    }
});

Beware: It doesn't supply parameters to the event.
I'v just place the code at the beginning of the view file.

[rdougan]

Can't reproduce in 2.0.1 final. Please repost if it is still an issue for you.

Thanks!

[yury.sannikov]

I'm not planning to switch to 2.0.1 before release. But, I'v reviewed the source code and it seems, it should work well.
Thank you for your business!

[rdougan]

Sencha Touch 2.0.1 is already final FYI: http://www.sencha.com/forum/announcement.php?f=92&a=30

[yury.sannikov]

Yes, I see. I mean our product release. Thanks.

[YomeKitsuma]

Temporary workaround that works for me is:

Ext.define('your.name.space', {
    override: 'Ext.field.Toggle',
    onComponentTap: function() {
        this.callParent(arguments);
        this.fireEvent('change');
        return false;
    }
});

Beware: It doesn't supply parameters to the event.
I'v just place the code at the beginning of the view file.

Juste add 'this' to get the Component =D

Ext.define('your.name.space', {    override: 'Ext.field.Toggle',
    onComponentTap: function() {
        this.callParent(arguments);
        this.fireEvent('change', this);
        return false;
    }
});