search

AmpersandJS / ampersand-select-view

Select field for ampersand-form-views
MIT License
11 stars 26 forks source link

onChange #54

Closed gnimmelf closed 9 years ago

gnimmelf commented 9 years ago

Hi,

ref https://github.com/AmpersandJS/ampersand-select-view/issues/11

there is no onChange event apart from that on the element.

There should be a change-event triggered on the view.

As of now, when extending, the onChange-method is un-overrridable (propbably due to the binding to this at construction?), and this.el is not avavilable in initialize, so hooking into the change-event is really cumbersome.

-Maybe I don't see the full picture, but this does not behave as I'd expect from working with the other AmpersandJs modules.

Cheers.

gnimmelf commented 9 years ago

My bad, just didn't catch the event, maybe a typo. Seems to trigger good now.