diff options
author | GeoSot <geo.sotis@gmail.com> | 2022-06-27 12:43:11 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-06-27 12:43:11 +0300 |
commit | f14c58f49471e7880cb8a13c74d88febc1c0c21d (patch) | |
tree | 6f5ba5a1fae02fd29267b223d9dbf8b2aceda039 /js/src | |
parent | 8de048833049f8b6af2c07df2d44d98c1ee095db (diff) |
ref(tab): remove show from panels dependent-less they have `.fade` class or not (simplify checks) (#36622)
Diffstat (limited to 'js/src')
-rw-r--r-- | js/src/tab.js | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/js/src/tab.js b/js/src/tab.js index 74bfc41484..9fad63e920 100644 --- a/js/src/tab.js +++ b/js/src/tab.js @@ -109,13 +109,9 @@ class Tab extends BaseComponent { this._activate(getElementFromSelector(element)) // Search and activate/show the proper section - const isAnimated = element.classList.contains(CLASS_NAME_FADE) const complete = () => { - if (isAnimated) { // todo: maybe is redundant - element.classList.add(CLASS_NAME_SHOW) - } - if (element.getAttribute('role') !== 'tab') { + element.classList.add(CLASS_NAME_SHOW) return } @@ -128,7 +124,7 @@ class Tab extends BaseComponent { }) } - this._queueCallback(complete, element, isAnimated) + this._queueCallback(complete, element, element.classList.contains(CLASS_NAME_FADE)) } _deactivate(element, relatedElem) { @@ -141,13 +137,9 @@ class Tab extends BaseComponent { this._deactivate(getElementFromSelector(element)) // Search and deactivate the shown section too - const isAnimated = element.classList.contains(CLASS_NAME_FADE) const complete = () => { - if (isAnimated) { // todo maybe is redundant - element.classList.remove(CLASS_NAME_SHOW) - } - if (element.getAttribute('role') !== 'tab') { + element.classList.remove(CLASS_NAME_SHOW) return } @@ -157,7 +149,7 @@ class Tab extends BaseComponent { EventHandler.trigger(element, EVENT_HIDDEN, { relatedTarget: relatedElem }) } - this._queueCallback(complete, element, isAnimated) + this._queueCallback(complete, element, element.classList.contains(CLASS_NAME_FADE)) } _keydown(event) { |