dmitriyakkerman / arm-ui

src/js/modules/tabs.js

"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
const globals_1 = require("../globals/globals");
(function (root, factory) {
    if (typeof globals_1.define === 'function' && globals_1.define.amd) {
        globals_1.define([], factory);
    }
    else if (typeof module === 'object' && module.exports) {
        module.exports = factory();
    }
    else {
        root.Tabs = factory();
    }
}(typeof self !== 'undefined' ? self : this, function () {

The variable self seems to be never declared. If this is a global, consider adding a /** global: self */ comment.

    class Tabs {
        constructor(options) {
            this.options = options || {};
            this.options.tabTogglers = (Object.keys(this.options).length && options.tabTogglers) ? (typeof options.tabTogglers === 'string' ? document.querySelectorAll(options.tabTogglers) : options.tabTogglers) : document.querySelectorAll('.tabs [data-pane]');
            this.onInit();
            this.onLoad();
        }
        static clearClasses(arr) {
            arr.forEach(function (el) {
                el.classList.remove('active');
            });
        }
        onLoad() {
            if (this.options && this.options.onLoad) {
                this.options.onLoad();
            }
        }
        onInit() {
            this.setCurrentOnInit();
            this.initClasses();
            this.changeCurrent();
        }
        setCurrentOnInit() {
            let currentTabToggler = (this.options.tabTogglers)[0];
            let currentPane;
            if (this.options.tabTogglers[0] instanceof HTMLElement) {

The variable HTMLElement seems to be never declared. If this is a global, consider adding a /** global: HTMLElement */ comment.

                currentPane = document.getElementById(currentTabToggler.dataset.pane);
            }
            currentTabToggler.classList.add('active');
            currentPane.classList.add('active');

The variable currentPane does not seem to be initialized in case this.options.tabTogglers.0 instanceof HTMLElement on line 40 is false. Are you sure this can never be the case?

        }
        initClasses() {
            this.options.tabTogglers.forEach(function (tabToggler) {
                let matchedPane = document.getElementById(tabToggler.dataset.pane);
                matchedPane.classList.add('tab-pane');
            });
        }
        changeCurrent() {
            let that = this;
            that.options.tabTogglers.forEach(function (tabToggler) {
                tabToggler.addEventListener('click', function (e) {
                    e.preventDefault();
                    let parentEl = this.closest('.tabs').parentElement;
                    Tabs.clearClasses(that.options.tabTogglers);
                    Tabs.clearClasses(parentEl.querySelectorAll('.tab-pane'));
                    this.classList.add('active');
                    let matchedPane = parentEl.querySelector('#' + this.dataset.pane);
                    matchedPane.classList.add('active');
                });
            });
        }
    }
    return Tabs;
}));