Collapse - Fix check to not prevent event for input and textarea

diff --git a/js/src/collapse.js b/js/src/collapse.js
index dc2e2a67d7d83c4f417ab8e343e60518bd37bd31..88428310d3943442f9d966224dbad507239a648b 100644 (file)
--- a/js/src/collapse.js
+++ b/js/src/collapse.js
@@ -357,7 +357,7 @@ const Collapse = (($) => {
    */
 
   $(document).on(Event.CLICK_DATA_API, Selector.DATA_TOGGLE, function (event) {
-    if (/input|textarea/i.test(event.target.tagName)) {
+    if (!/input|textarea/i.test(event.target.tagName)) {
       event.preventDefault()
     }
 

diff --git a/js/tests/unit/collapse.js b/js/tests/unit/collapse.js
index 01bc5adb96a7597044795f4ac60a0c75745dedac..e7083f56ddc9964248d60358d5d7d0c2dd0aee62 100644 (file)
--- a/js/tests/unit/collapse.js
+++ b/js/tests/unit/collapse.js
@@ -515,13 +515,14 @@ $(function () {
   })
 
   QUnit.test('should not prevent event for input', function (assert) {
-    assert.expect(2)
+    assert.expect(3)
     var done = assert.async()
     var $target = $('<input type="checkbox" data-toggle="collapse" data-target="#collapsediv1" />').appendTo('#qunit-fixture')
 
     $('<div id="collapsediv1"/>')
       .appendTo('#qunit-fixture')
       .on('shown.bs.collapse', function () {
+        assert.ok($(this).hasClass('show'))
         assert.ok($target.attr('aria-expanded') === 'true')
         assert.ok($target.prop('checked'))
         done()