From ed94976ea80cfe5fa2bd9b6a6913c78de8626045 Mon Sep 17 00:00:00 2001 From: XhmikosR Date: Sun, 29 Mar 2020 09:53:57 +0300 Subject: Use `textContent` instead of `innerText`. (#30462) It's supported by all of our supported browsers. --- js/src/tooltip.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'js/src') diff --git a/js/src/tooltip.js b/js/src/tooltip.js index c493b15ee..9a1dff3d6 100644 --- a/js/src/tooltip.js +++ b/js/src/tooltip.js @@ -421,7 +421,7 @@ class Tooltip { element.appendChild(content) } } else { - element.innerText = content.textContent + element.textContent = content.textContent } return @@ -434,7 +434,7 @@ class Tooltip { element.innerHTML = content } else { - element.innerText = content + element.textContent = content } } -- cgit v1.2.3 From e1f5d819c73ad66e6ec0480e75e5e08c815a633e Mon Sep 17 00:00:00 2001 From: XhmikosR Date: Sun, 29 Mar 2020 09:59:54 +0300 Subject: Remove `MSPointerEvent` and `ms-touch-action` in tests (#30463) --- js/src/carousel.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'js/src') diff --git a/js/src/carousel.js b/js/src/carousel.js index 509f7ca9d..3020974d3 100644 --- a/js/src/carousel.js +++ b/js/src/carousel.js @@ -118,7 +118,7 @@ class Carousel { this._element = element this._indicatorsElement = SelectorEngine.findOne(SELECTOR_INDICATORS, this._element) this._touchSupported = 'ontouchstart' in document.documentElement || navigator.maxTouchPoints > 0 - this._pointerEvent = Boolean(window.PointerEvent || window.MSPointerEvent) + this._pointerEvent = Boolean(window.PointerEvent) this._addEventListeners() Data.setData(element, DATA_KEY, this) -- cgit v1.2.3 From dfa017adc382c63766cd5d021bdc8f6ad1b04932 Mon Sep 17 00:00:00 2001 From: ysds Date: Wed, 4 Mar 2020 23:51:15 +0900 Subject: Always modal scroll top to 0 --- js/src/modal.js | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'js/src') diff --git a/js/src/modal.js b/js/src/modal.js index ca77359ca..d5f1c8c9a 100644 --- a/js/src/modal.js +++ b/js/src/modal.js @@ -60,7 +60,6 @@ const EVENT_MOUSEUP_DISMISS = `mouseup.dismiss${EVENT_KEY}` const EVENT_MOUSEDOWN_DISMISS = `mousedown.dismiss${EVENT_KEY}` const EVENT_CLICK_DATA_API = `click${EVENT_KEY}${DATA_API_KEY}` -const CLASS_NAME_SCROLLABLE = 'modal-dialog-scrollable' const CLASS_NAME_SCROLLBAR_MEASURER = 'modal-scrollbar-measure' const CLASS_NAME_BACKDROP = 'modal-backdrop' const CLASS_NAME_OPEN = 'modal-open' @@ -249,11 +248,10 @@ class Modal { this._element.style.display = 'block' this._element.removeAttribute('aria-hidden') this._element.setAttribute('aria-modal', true) + this._element.scrollTop = 0 - if (this._dialog.classList.contains(CLASS_NAME_SCROLLABLE) && modalBody) { + if (modalBody) { modalBody.scrollTop = 0 - } else { - this._element.scrollTop = 0 } if (transition) { -- cgit v1.2.3 From 9a5033d41350f83d3b4814648934618ab4648e27 Mon Sep 17 00:00:00 2001 From: XhmikosR Date: Thu, 9 Apr 2020 20:56:43 +0300 Subject: Pass multiple classes to `classList.add` / `classList.remove` (#30537) This is supported by our currently supported browsers. --- js/src/carousel.js | 7 ++----- js/src/collapse.js | 6 ++---- js/src/popover.js | 3 +-- js/src/tooltip.js | 3 +-- 4 files changed, 6 insertions(+), 13 deletions(-) (limited to 'js/src') diff --git a/js/src/carousel.js b/js/src/carousel.js index 3020974d3..a5451538c 100644 --- a/js/src/carousel.js +++ b/js/src/carousel.js @@ -478,13 +478,10 @@ class Carousel { EventHandler .one(activeElement, TRANSITION_END, () => { - nextElement.classList.remove(directionalClassName) - nextElement.classList.remove(orderClassName) + nextElement.classList.remove(directionalClassName, orderClassName) nextElement.classList.add(CLASS_NAME_ACTIVE) - activeElement.classList.remove(CLASS_NAME_ACTIVE) - activeElement.classList.remove(orderClassName) - activeElement.classList.remove(directionalClassName) + activeElement.classList.remove(CLASS_NAME_ACTIVE, orderClassName, directionalClassName) this._isSliding = false diff --git a/js/src/collapse.js b/js/src/collapse.js index fde3e0a9e..4529a85ec 100644 --- a/js/src/collapse.js +++ b/js/src/collapse.js @@ -192,8 +192,7 @@ class Collapse { const complete = () => { this._element.classList.remove(CLASS_NAME_COLLAPSING) - this._element.classList.add(CLASS_NAME_COLLAPSE) - this._element.classList.add(CLASS_NAME_SHOW) + this._element.classList.add(CLASS_NAME_COLLAPSE, CLASS_NAME_SHOW) this._element.style[dimension] = '' @@ -230,8 +229,7 @@ class Collapse { reflow(this._element) this._element.classList.add(CLASS_NAME_COLLAPSING) - this._element.classList.remove(CLASS_NAME_COLLAPSE) - this._element.classList.remove(CLASS_NAME_SHOW) + this._element.classList.remove(CLASS_NAME_COLLAPSE, CLASS_NAME_SHOW) const triggerArrayLength = this._triggerArray.length if (triggerArrayLength > 0) { diff --git a/js/src/popover.js b/js/src/popover.js index 2d423f0dd..4aac7e8ab 100644 --- a/js/src/popover.js +++ b/js/src/popover.js @@ -113,8 +113,7 @@ class Popover extends Tooltip { this.setElementContent(SelectorEngine.findOne(SELECTOR_CONTENT, tip), content) - tip.classList.remove(CLASS_NAME_FADE) - tip.classList.remove(CLASS_NAME_SHOW) + tip.classList.remove(CLASS_NAME_FADE, CLASS_NAME_SHOW) } _addAttachmentClass(attachment) { diff --git a/js/src/tooltip.js b/js/src/tooltip.js index 9a1dff3d6..3f40b3da1 100644 --- a/js/src/tooltip.js +++ b/js/src/tooltip.js @@ -400,8 +400,7 @@ class Tooltip { setContent() { const tip = this.getTipElement() this.setElementContent(SelectorEngine.findOne(SELECTOR_TOOLTIP_INNER, tip), this.getTitle()) - tip.classList.remove(CLASS_NAME_FADE) - tip.classList.remove(CLASS_NAME_SHOW) + tip.classList.remove(CLASS_NAME_FADE, CLASS_NAME_SHOW) } setElementContent(element, content) { -- cgit v1.2.3