Event emitter hook for every events #171

Yomguithereal · Aug 26, 2015

Hello @chjj, would you be willing to consider to extend the blessed event emitter to accept a listener handling every events? Something of this kind:

node.onAny(function(type, payload) {});

chjj · Aug 28, 2015

Yeah, I would consider that. I'd rather make it special event though: element.on('emit') or element.on('event'). The only thing I worry about is the bubbling. It might be a hit to performance if these events hve to bubble along with the actual event. We'll see. Maybe I'm just being paranoid, or we could just have it not bubble.

Yomguithereal · Aug 31, 2015

Adding an event event seems to be a good and simple way to do so even if this feels a bit meta :).

To add some background to this issue this is where I monkey patch the blessed emitter to be able to catch events efficiently with react-blessed.

Yomguithereal · Sep 3, 2015

Thanks @chjj. Will switch to 0.1.81 asap.

chjj added a commit that closed this issue Sep 3, 2015

chjj add "event" for any event. fixes #171. e4900bb

chjj closed this in e4900bb Sep 3, 2015

Yomguithereal referenced this issue in Yomguithereal/react-blessed Sep 3, 2015
Closed
Switch to blessed 0.1.81 #15

chjj/blessed

Event emitter hook for every events #171

Assignees

Labels

Projects

Milestone

2 participants

Switch to blessed 0.1.81 #15