search

medikoo / event-emitter

Environment agnostic event emitter solution for JavaScript
MIT License
241 stars 23 forks source link

Clarify mixin support in readme #12

Closed sminnee closed 8 years ago

sminnee commented 8 years ago

Use as a mix-in appears to be a supported mode of use, but it wasn't apparent to me until I read the code. This adds a small example to the readme.

If this isn't an intended use-case, it would be good to understand how this package can be used as a mix-in.

medikoo commented 8 years ago

@sminnee I've found example that you've added controversial, still I've updated documentation with your suggestion put slightly different way.

If you still feel something should be improved, let me know

sminnee commented 8 years ago

OK, that's great, thanks! I wasn't sure if a ee() call per object was necessary, but your approach looks good.

I'll go and refactor my own code now to match the example :-)