diff --git a/js/src/button.js b/js/src/button.js index 6edd5cb64f..314d2a22a0 100644 --- a/js/src/button.js +++ b/js/src/button.js @@ -25,6 +25,7 @@ const DATA_API_KEY = '.data-api' const ClassName = { ACTIVE: 'active', BUTTON: 'btn', + DISABLED: 'disabled', FOCUS: 'focus' } @@ -89,8 +90,8 @@ class Button { if (triggerChangeEvent) { if (input.hasAttribute('disabled') || rootElement.hasAttribute('disabled') || - input.classList.contains('disabled') || - rootElement.classList.contains('disabled')) { + input.classList.contains(ClassName.DISABLED) || + rootElement.classList.contains(ClassName.DISABLED)) { return } diff --git a/js/src/collapse.js b/js/src/collapse.js index f533885ec4..27eef4ef72 100644 --- a/js/src/collapse.js +++ b/js/src/collapse.js @@ -81,8 +81,8 @@ class Collapse { this._element = element this._config = this._getConfig(config) this._triggerArray = makeArray(SelectorEngine.find( - `[data-toggle="collapse"][href="#${element.id}"],` + - `[data-toggle="collapse"][data-target="#${element.id}"]` + `${Selector.DATA_TOGGLE}[href="#${element.id}"],` + + `${Selector.DATA_TOGGLE}[data-target="#${element.id}"]` )) const toggleList = makeArray(SelectorEngine.find(Selector.DATA_TOGGLE)) @@ -313,7 +313,7 @@ class Collapse { parent = SelectorEngine.findOne(parent) } - const selector = `[data-toggle="collapse"][data-parent="${parent}"]` + const selector = `${Selector.DATA_TOGGLE}[data-parent="${parent}"]` makeArray(SelectorEngine.find(selector, parent)) .forEach(element => { diff --git a/js/src/dropdown.js b/js/src/dropdown.js index 4504d61e09..82a24405aa 100644 --- a/js/src/dropdown.js +++ b/js/src/dropdown.js @@ -57,6 +57,7 @@ const ClassName = { DROPRIGHT: 'dropright', DROPLEFT: 'dropleft', MENURIGHT: 'dropdown-menu-right', + NAVBAR: 'navbar', POSITION_STATIC: 'position-static' } @@ -307,7 +308,7 @@ class Dropdown { } _detectNavbar() { - return Boolean(SelectorEngine.closest(this._element, '.navbar')) + return Boolean(SelectorEngine.closest(this._element, `.${ClassName.NAVBAR}`)) } _getOffset() { diff --git a/js/src/tooltip.js b/js/src/tooltip.js index 1bc000d2cc..9b2ae6f61e 100644 --- a/js/src/tooltip.js +++ b/js/src/tooltip.js @@ -109,6 +109,7 @@ const Event = { const ClassName = { FADE: 'fade', + MODAL: 'modal', SHOW: 'show' } @@ -235,7 +236,7 @@ class Tooltip { Data.removeData(this.element, this.constructor.DATA_KEY) EventHandler.off(this.element, this.constructor.EVENT_KEY) - EventHandler.off(SelectorEngine.closest(this.element, '.modal'), 'hide.bs.modal', this._hideModalHandler) + EventHandler.off(SelectorEngine.closest(this.element, `.${ClassName.MODAL}`), 'hide.bs.modal', this._hideModalHandler) if (this.tip) { this.tip.parentNode.removeChild(this.tip) @@ -565,7 +566,7 @@ class Tooltip { } } - EventHandler.on(SelectorEngine.closest(this.element, '.modal'), + EventHandler.on(SelectorEngine.closest(this.element, `.${ClassName.MODAL}`), 'hide.bs.modal', this._hideModalHandler )