ng-inspector icon indicating copy to clipboard operation
ng-inspector copied to clipboard

Use event delegates for interaction with the panel UI

Open rev087 opened this issue 9 years ago • 2 comments

Currently, each clickable element in the inspector UI adds it's own event listener. An optimization opportunity here is to replace this mechanic with a single delegate event on the parent node.

This would also fix the second half of #113.

rev087 avatar May 26 '15 13:05 rev087

Note: The second-half of #113 only requires mouseout and mouseover events to be delegated.

I totally agree we should address our over-use of event handlers in every scenario. Having said that, it'll get us back into the Firefox store quicker if we just address mouseout and mouseover for now, and open a separate issue for other events (like clicks). It looks like the mouseout and mouseover events will be quicker to refactor than the click events, the way everything is currently written

DrewML avatar Sep 13 '15 19:09 DrewML

Related PR submitted in #137

DrewML avatar Sep 27 '15 19:09 DrewML