diff --git a/js/src/carousel.js b/js/src/carousel.js index 6443ef094a7f3f517b7238d280683b98de0c7a64..9c6fb53ee0428fa2d8a1e6d637c9b65c980d4c14 100644 --- a/js/src/carousel.js +++ b/js/src/carousel.js @@ -397,6 +397,7 @@ class Carousel extends BaseComponent { _setActiveIndicatorElement(element) { if (this._indicatorsElement) { const indicators = SelectorEngine.find(SELECTOR_ACTIVE, this._indicatorsElement) + for (let i = 0; i < indicators.length; i++) { indicators[i].classList.remove(CLASS_NAME_ACTIVE) } @@ -431,8 +432,7 @@ class Carousel extends BaseComponent { _slide(direction, element) { const activeElement = SelectorEngine.findOne(SELECTOR_ACTIVE_ITEM, this._element) const activeElementIndex = this._getItemIndex(activeElement) - const nextElement = element || (activeElement && - this._getItemByDirection(direction, activeElement)) + const nextElement = element || (activeElement && this._getItemByDirection(direction, activeElement)) const nextElementIndex = this._getItemIndex(nextElement) const isCycling = Boolean(this._interval) diff --git a/js/src/collapse.js b/js/src/collapse.js index feff6c9178e4bb76f98ef6e793408243e384c0ee..0ad2b198e992c97bcd3c4e10ff1af97ccd512686 100644 --- a/js/src/collapse.js +++ b/js/src/collapse.js @@ -124,8 +124,7 @@ class Collapse extends BaseComponent { } show() { - if (this._isTransitioning || - this._element.classList.contains(CLASS_NAME_SHOW)) { + if (this._isTransitioning || this._element.classList.contains(CLASS_NAME_SHOW)) { return } @@ -212,8 +211,7 @@ class Collapse extends BaseComponent { } hide() { - if (this._isTransitioning || - !this._element.classList.contains(CLASS_NAME_SHOW)) { + if (this._isTransitioning || !this._element.classList.contains(CLASS_NAME_SHOW)) { return } diff --git a/js/src/dropdown.js b/js/src/dropdown.js index 5b8ae064542db3db73f333ffe0746311754aaea1..7b3bf5b4ec3fc909b4a1318dbdfcf5fc579d61a2 100644 --- a/js/src/dropdown.js +++ b/js/src/dropdown.js @@ -367,8 +367,7 @@ class Dropdown extends BaseComponent { } static clearMenus(event) { - if (event && (event.button === RIGHT_MOUSE_BUTTON || - (event.type === 'keyup' && event.key !== TAB_KEY))) { + if (event && (event.button === RIGHT_MOUSE_BUTTON || (event.type === 'keyup' && event.key !== TAB_KEY))) { return } @@ -475,11 +474,13 @@ class Dropdown extends BaseComponent { let index = items.indexOf(event.target) - if (event.key === ARROW_UP_KEY && index > 0) { // Up + // Up + if (event.key === ARROW_UP_KEY && index > 0) { index-- } - if (event.key === ARROW_DOWN_KEY && index < items.length - 1) { // Down + // Down + if (event.key === ARROW_DOWN_KEY && index < items.length - 1) { index++ } diff --git a/js/src/modal.js b/js/src/modal.js index fdca482135f59b8f3b6171ee542815cb29b45b12..a9cf8ae6cff7cbd4cdf15dabd2189c4996ef7b54 100644 --- a/js/src/modal.js +++ b/js/src/modal.js @@ -136,11 +136,7 @@ class Modal extends BaseComponent { this._setEscapeEvent() this._setResizeEvent() - EventHandler.on(this._element, - EVENT_CLICK_DISMISS, - SELECTOR_DATA_DISMISS, - event => this.hide(event) - ) + EventHandler.on(this._element, EVENT_CLICK_DISMISS, SELECTOR_DATA_DISMISS, event => this.hide(event)) EventHandler.on(this._dialog, EVENT_MOUSEDOWN_DISMISS, () => { EventHandler.one(this._element, EVENT_MOUSEUP_DISMISS, event => { @@ -237,8 +233,7 @@ class Modal extends BaseComponent { const transition = this._element.classList.contains(CLASS_NAME_FADE) const modalBody = SelectorEngine.findOne(SELECTOR_MODAL_BODY, this._dialog) - if (!this._element.parentNode || - this._element.parentNode.nodeType !== Node.ELEMENT_NODE) { + if (!this._element.parentNode || this._element.parentNode.nodeType !== Node.ELEMENT_NODE) { // Don't move modal's DOM position document.body.appendChild(this._element) } diff --git a/js/src/popover.js b/js/src/popover.js index 97db9a3f7d2a5d324a76ec414cb2d383c697277f..66dcb47b90031d7c5237dd9fb9688b4884931a04 100644 --- a/js/src/popover.js +++ b/js/src/popover.js @@ -29,8 +29,9 @@ const Default = { content: '', template: '<div class="popover" role="tooltip">' + '<div class="popover-arrow"></div>' + - '<h3 class="popover-header"></h3>' + - '<div class="popover-body"></div></div>' + '<h3 class="popover-header"></h3>' + + '<div class="popover-body"></div>' + + '</div>' } const DefaultType = { @@ -118,8 +119,7 @@ class Popover extends Tooltip { } _getContent() { - return this._element.getAttribute('data-bs-content') || - this.config.content + return this._element.getAttribute('data-bs-content') || this.config.content } _cleanTipClass() { diff --git a/js/src/scrollspy.js b/js/src/scrollspy.js index e3e5e76b97d45763d7c134b235eb705dffdd0a1b..a05e57d623ed0afe55eaca6a3c64f7944c3f37b3 100644 --- a/js/src/scrollspy.js +++ b/js/src/scrollspy.js @@ -196,9 +196,7 @@ class ScrollSpy extends BaseComponent { _process() { const scrollTop = this._getScrollTop() + this._config.offset const scrollHeight = this._getScrollHeight() - const maxScroll = this._config.offset + - scrollHeight - - this._getOffsetHeight() + const maxScroll = this._config.offset + scrollHeight - this._getOffsetHeight() if (this._scrollHeight !== scrollHeight) { this.refresh() @@ -223,8 +221,7 @@ class ScrollSpy extends BaseComponent { for (let i = this._offsets.length; i--;) { const isActiveTarget = this._activeTarget !== this._targets[i] && scrollTop >= this._offsets[i] && - (typeof this._offsets[i + 1] === 'undefined' || - scrollTop < this._offsets[i + 1]) + (typeof this._offsets[i + 1] === 'undefined' || scrollTop < this._offsets[i + 1]) if (isActiveTarget) { this._activate(this._targets[i]) diff --git a/js/src/tab.js b/js/src/tab.js index 0e9755ea026425c1a38d629faad1a989d8f9d524..c8aac3be724464f218c332b0e5da2e6c81eac0e6 100644 --- a/js/src/tab.js +++ b/js/src/tab.js @@ -95,15 +95,11 @@ class Tab extends BaseComponent { relatedTarget: previous }) - if (showEvent.defaultPrevented || - (hideEvent !== null && hideEvent.defaultPrevented)) { + if (showEvent.defaultPrevented || (hideEvent !== null && hideEvent.defaultPrevented)) { return } - this._activate( - this._element, - listElement - ) + this._activate(this._element, listElement) const complete = () => { EventHandler.trigger(previous, EVENT_HIDDEN, { @@ -129,14 +125,9 @@ class Tab extends BaseComponent { SelectorEngine.children(container, SELECTOR_ACTIVE) const active = activeElements[0] - const isTransitioning = callback && - (active && active.classList.contains(CLASS_NAME_FADE)) - - const complete = () => this._transitionComplete( - element, - active, - callback - ) + const isTransitioning = callback && (active && active.classList.contains(CLASS_NAME_FADE)) + + const complete = () => this._transitionComplete(element, active, callback) if (active && isTransitioning) { const transitionDuration = getTransitionDurationFromElement(active) diff --git a/js/src/tooltip.js b/js/src/tooltip.js index 62b33883988c5c0282fbc35a09450ae3e735fe11..25599bb42f8f0a53191c46187ea5633ee8d0cc38 100644 --- a/js/src/tooltip.js +++ b/js/src/tooltip.js @@ -72,8 +72,9 @@ const AttachmentMap = { const Default = { animation: true, template: '<div class="tooltip" role="tooltip">' + - '<div class="tooltip-arrow"></div>' + - '<div class="tooltip-inner"></div></div>', + '<div class="tooltip-arrow"></div>' + + '<div class="tooltip-inner"></div>' + + '</div>', trigger: 'hover focus', title: '', delay: 0, @@ -196,10 +197,7 @@ class Tooltip extends BaseComponent { let context = Data.getData(event.delegateTarget, dataKey) if (!context) { - context = new this.constructor( - event.delegateTarget, - this._getDelegateConfig() - ) + context = new this.constructor(event.delegateTarget, this._getDelegateConfig()) Data.setData(event.delegateTarget, dataKey, context) } @@ -530,10 +528,7 @@ class Tooltip extends BaseComponent { triggers.forEach(trigger => { if (trigger === 'click') { - EventHandler.on(this._element, - this.constructor.Event.CLICK, - this.config.selector, - event => this.toggle(event) + EventHandler.on(this._element, this.constructor.Event.CLICK, this.config.selector, event => this.toggle(event) ) } else if (trigger !== TRIGGER_MANUAL) { const eventIn = trigger === TRIGGER_HOVER ? @@ -543,16 +538,8 @@ class Tooltip extends BaseComponent { this.constructor.Event.MOUSELEAVE : this.constructor.Event.FOCUSOUT - EventHandler.on(this._element, - eventIn, - this.config.selector, - event => this._enter(event) - ) - EventHandler.on(this._element, - eventOut, - this.config.selector, - event => this._leave(event) - ) + EventHandler.on(this._element, eventIn, this.config.selector, event => this._enter(event)) + EventHandler.on(this._element, eventOut, this.config.selector, event => this._leave(event)) } }) @@ -562,10 +549,7 @@ class Tooltip extends BaseComponent { } } - EventHandler.on(this._element.closest(`.${CLASS_NAME_MODAL}`), - 'hide.bs.modal', - this._hideModalHandler - ) + EventHandler.on(this._element.closest(`.${CLASS_NAME_MODAL}`), 'hide.bs.modal', this._hideModalHandler) if (this.config.selector) { this.config = { @@ -610,8 +594,7 @@ class Tooltip extends BaseComponent { ] = true } - if (context.getTipElement().classList.contains(CLASS_NAME_SHOW) || - context._hoverState === HOVER_STATE_SHOW) { + if (context.getTipElement().classList.contains(CLASS_NAME_SHOW) || context._hoverState === HOVER_STATE_SHOW) { context._hoverState = HOVER_STATE_SHOW return } diff --git a/js/src/util/index.js b/js/src/util/index.js index c7cb3176a4b9f16467bd1cf3e08cec0ad69fcb67..874827b16880c09c384d98bf33d61fbb8ea96d13 100644 --- a/js/src/util/index.js +++ b/js/src/util/index.js @@ -66,10 +66,7 @@ const getTransitionDurationFromElement = element => { } // Get transition-duration of the element - let { - transitionDuration, - transitionDelay - } = window.getComputedStyle(element) + let { transitionDuration, transitionDelay } = window.getComputedStyle(element) const floatTransitionDuration = Number.parseFloat(transitionDuration) const floatTransitionDelay = Number.parseFloat(transitionDelay) @@ -96,6 +93,7 @@ const emulateTransitionEnd = (element, duration) => { let called = false const durationPadding = 5 const emulatedDuration = duration + durationPadding + function listener() { called = true element.removeEventListener(TRANSITION_END, listener)