Use cached `noop` function everywhere

diff --git a/js/src/dropdown.js b/js/src/dropdown.js
index 6b541ed150b3dfb508a407c2205410758b8cb23d..3e1ef5fd4f5b007acd2a3dbe06fd9500b73dbb7f 100644 (file)
--- a/js/src/dropdown.js
+++ b/js/src/dropdown.js
@@ -192,7 +192,7 @@ class Dropdown extends BaseComponent {
     if ('ontouchstart' in document.documentElement &&
       !parent.closest(SELECTOR_NAVBAR_NAV)) {
       [].concat(...document.body.children)
-        .forEach(elem => EventHandler.on(elem, 'mouseover', null, noop()))
+        .forEach(elem => EventHandler.on(elem, 'mouseover', null, noop))
     }
 
     this._element.focus()
@@ -222,7 +222,7 @@ class Dropdown extends BaseComponent {
     // empty mouseover listeners we added for iOS support
     if ('ontouchstart' in document.documentElement) {
       [].concat(...document.body.children)
-        .forEach(elem => EventHandler.off(elem, 'mouseover', null, noop()))
+        .forEach(elem => EventHandler.off(elem, 'mouseover', null, noop))
     }
 
     if (this._popper) {
@@ -435,7 +435,7 @@ class Dropdown extends BaseComponent {
       // empty mouseover listeners we added for iOS support
       if ('ontouchstart' in document.documentElement) {
         [].concat(...document.body.children)
-          .forEach(elem => EventHandler.off(elem, 'mouseover', null, noop()))
+          .forEach(elem => EventHandler.off(elem, 'mouseover', null, noop))
       }
 
       if (context._popper) {

diff --git a/js/src/tooltip.js b/js/src/tooltip.js
index 4fea1c9646e3c38ff3f5cb9ad463b8debd789f0a..a66e1ad41e54b5a28c862aaaeaefd180bb276eb0 100644 (file)
--- a/js/src/tooltip.js
+++ b/js/src/tooltip.js
@@ -301,7 +301,7 @@ class Tooltip extends BaseComponent {
     // https://www.quirksmode.org/blog/archives/2014/02/mouse_event_bub.html
     if ('ontouchstart' in document.documentElement) {
       [].concat(...document.body.children).forEach(element => {
-        EventHandler.on(element, 'mouseover', noop())
+        EventHandler.on(element, 'mouseover', noop)
       })
     }
 

diff --git a/js/src/util/index.js b/js/src/util/index.js
index cc35d8a37c4c912fd75b042fb72c1f54dd7a7030..f19d76e036e917afa788d3cb184b6f9f6d04f14b 100644 (file)
--- a/js/src/util/index.js
+++ b/js/src/util/index.js
@@ -190,7 +190,7 @@ const findShadowRoot = element => {
   return findShadowRoot(element.parentNode)
 }
 
-const noop = () => function () {}
+const noop = () => {}
 
 const reflow = element => element.offsetHeight
 

diff --git a/js/tests/unit/util/index.spec.js b/js/tests/unit/util/index.spec.js
index 41c1ce2b80b44251ed441f8cc9e4b3b89e918fde..5d144348e458ca9b49cbd473daebbd77b9e5de79 100644 (file)
--- a/js/tests/unit/util/index.spec.js
+++ b/js/tests/unit/util/index.spec.js
@@ -477,8 +477,8 @@ describe('Util', () => {
   })
 
   describe('noop', () => {
-    it('should return a function', () => {
-      expect(typeof Util.noop()).toEqual('function')
+    it('should be a function', () => {
+      expect(typeof Util.noop).toEqual('function')
     })
   })