Use more safe check for 'isDisabled' helper (#33385)

diff --git a/.bundlewatch.config.json b/.bundlewatch.config.json
index 4a28264b3a9919ecf18668091bd217e3c3854690..27b998c6201a5969833264806b7de973c16420d1 100644 (file)
--- a/.bundlewatch.config.json
+++ b/.bundlewatch.config.json
@@ -54,7 +54,7 @@
     },
     {
       "path": "./dist/js/bootstrap.min.js",
-      "maxSize": "16.25 kB"
+      "maxSize": "16.5 kB"
     }
   ],
   "ci": {

diff --git a/js/src/util/index.js b/js/src/util/index.js
index e268b07287a07e88fda6b97007ead76373b91af4..e9950c9e387ab236f2739798be86f6fdac9a48a9 100644 (file)
--- a/js/src/util/index.js
+++ b/js/src/util/index.js
@@ -166,7 +166,7 @@ const isDisabled = element => {
     return element.disabled
   }
 
-  return element.getAttribute('disabled') !== 'false'
+  return element.hasAttribute('disabled') && element.getAttribute('disabled') !== 'false'
 }
 
 const findShadowRoot = element => {

diff --git a/js/tests/unit/util/index.spec.js b/js/tests/unit/util/index.spec.js
index 24921d730e7b84bdc385ac99b1a1ec6d5e70495a..41c1ce2b80b44251ed441f8cc9e4b3b89e918fde 100644 (file)
--- a/js/tests/unit/util/index.spec.js
+++ b/js/tests/unit/util/index.spec.js
@@ -347,16 +347,19 @@ describe('Util', () => {
       expect(Util.isDisabled(div2)).toEqual(true)
     })
 
-    it('should return false if the element has disabled attribute with "false" value', () => {
+    it('should return false if the element has disabled attribute with "false" value, or doesn\'t have attribute', () => {
       fixtureEl.innerHTML = [
         '<div>',
         '  <div id="element" disabled="false"></div>',
+        '  <div id="element1" ></div>',
         '</div>'
       ].join('')
 
       const div = fixtureEl.querySelector('#element')
+      const div1 = fixtureEl.querySelector('#element1')
 
       expect(Util.isDisabled(div)).toEqual(false)
+      expect(Util.isDisabled(div1)).toEqual(false)
     })
 
     it('should return false if the element is not disabled ', () => {