metafizzy / outlayer

:construction_worker: the brains & guts of a layout library
163 stars 63 forks source link

EventEmitter is too heavy, 15 KB! #11

Closed binarykitchen closed 10 years ago

binarykitchen commented 10 years ago

Consider replacing it with a much smaller one: https://github.com/thomaswelton/bower-event-emitter

It's 2 KB only and is really good enough ...

desandro commented 10 years ago

Interesting proposal! :sparkles:

15KB is unminified. And the majority of that is comments.

Pros

Cons

As I see it, it's not worth switching over for a 3kb filesize benefit. I appreciate your interest in the library, but I think there are more effective gains to be made elsewhere.

binarykitchen commented 10 years ago

I understand your comments. But when I aim to port my single page app with PhoneGap to a native iPhone app, then every kilobyte does matter!

There are too many event emitter implementations out there and this is a pain in the arse. I already have three different event emitters in my bower directory. They all implement the same functionality. The wheel has been reinvented too many times, bloating up stuff.