fix(stateDirective): using on to attach an event handler instead of bind

[navarroaxel]

fix #3035

[nateabele]

This is a BC-break. We can add support for on and off, but not by dropping support for bind/unbind.

[navarroaxel]

Yes @nateabele, it's a breaker.
How do you like the IF?

element.bind ? element.bind("click", hookFn) : element.on("click", hookFn);

or this:

element.[element.bind ? 'bind' : 'on']("click", hookFn);

Should I try to use on before bind or viceversa?

[christopherthielen]

I think I merged a similar commit into master

[christopherthielen]

@navarroaxel we would want to try on before bind

[navarroaxel]

Hey @christopherthielen, are you saying this commit is useless because it was covered by another one?

[navarroaxel]

Hey @nateabele, I have change the code to be keep the BC.

[christopherthielen]

I think I merged a similar commit on master, not legacy

[christopherthielen]

Thanks!

[nateabele]

Yup, thanks @navarroaxel.