fix(bind): Fix eventname and controller binding bugs.

[latentflip]

This is a quick PR to fix

• data-action event names cannot contain : #29 - event names not being able to contain :
• data-action does not work if added to the controller's element #30 - not being able to add data-action to the controller itself

I'm not 100% in love with the implementation here. I broke out bindActionsToController to avoid doing something like this:

for (const el of [this, ...this.querySelectorAll(`[data-action*=":${tag}#"]`)]) {

but that's resulted in quite a hefty diff. I broke it up into clear commits for review, but maybe there's a better-better way I've missed?

[koddsson]

I think allowing : in a event name is great and a oversight on our part so that's great to get fixed.

I kind of like the approach of this for loop:

for (const el of [this, ...this.querySelectorAll(`[data-action*=":${tag}#"]`)]) {

rather than creating a bindActionsToController function but that might just be a personal preference.

In any case this looks good to me and I'm not overly fuzzed about coding style.

[keithamus]

This works well, I think either pattern is good 👍