firing legacy change event #3

Merged
merged 1 commit into from Dec 7, 2012

Conversation

Projects
None yet
2 participants
Contributor

backbone87 commented Dec 3, 2012

No description provided.

Great, I will merge for sure, but...
Why this.element.getParents().fireEvent('change');?

Owner

backbone87 replied Dec 5, 2012

Yes, but why you have to bubble up the change event? Won't you bound events on the select element itself?

Owner

backbone87 replied Dec 6, 2012

Not in every case.
A filter form for example, which gets automatically send via AJAX to update filtered list. In this case it easier to listen on the form itself for every change event.

@lorenzos lorenzos added a commit that referenced this pull request Dec 7, 2012

@lorenzos lorenzos Merge pull request #3 from backbone87/master
firing legacy change event
de41bd7

@lorenzos lorenzos merged commit de41bd7 into lorenzos:master Dec 7, 2012

Owner

lorenzos commented Dec 7, 2012

Merged :)

@lorenzos lorenzos added a commit that referenced this pull request Dec 7, 2012

@lorenzos lorenzos Fixes for pull request #3 2375082
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment