Tabs component binds the same method multiple times

[romaricpascal]

What

The tabs component binds the onTabClick and onTabKeydown listeners multiple times, while only a single version of that bound method is necessary.

Furthermore, it stores each bound version on the $tab element, poluting the object from the DOM.

These methods could be bound only once before looping and stored on the component instead.

Why

Avoid unnecessary extra functions being created and poluting objects from the browser.

Who needs to work on this

Developers

Who needs to review this

Developers

Done when

• The method is bound prior to looping and stored on the component rather than the element

[36degrees]

Fixed by #2987 (478f642)