bootstrap-input-spinner 5.0.6 → 5.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bootstrap-input-spinner",
-  "version": "5.0.6",
+  "version": "5.0.7",
   "description": "A Bootstrap 5 plugin to create input spinner elements for number input.",
   "browser": "./src/InputSpinner.js",
   "type": "module",

package/src/InputSpinner.js

@@ -46,8 +46,6 @@ const I18nEditor = function (props, element) {
     }
 }
 
-let triggerKeyPressed = false
-
 function parseTemplate(html) {
     const tpl = document.createElement("template")
     tpl.innerHTML = html.trim()
@@ -159,6 +157,8 @@ export class InputSpinner {
             destroy()
         }
 
+        let triggerKeyPressed = false
+
         this.observer = new MutationObserver(function () {
             updateAttributes()
             setValue(self.value, true)
@@ -364,9 +364,9 @@ export class InputSpinner {
             }
             const originalClass = self.original.className || ""
             let groupClass = ""
-            if (/form-control-sm/g.test(originalClass)) {
+            if (/form-control-sm/.test(originalClass)) {
                 groupClass = "input-group-sm"
-            } else if (/form-control-lg/g.test(originalClass)) {
+            } else if (/form-control-lg/.test(originalClass)) {
                 groupClass = "input-group-lg"
             }
             const inputClass = originalClass.replace(/form-control(-(sm|lg))?/g, "")

package/test/TestInputSpinner.js

@@ -341,6 +341,28 @@ describe("InputSpinner instance isolation", () => {
         assert.equal(b, 0)
         clear()
     })
+    it("two spinners both respond to keyboard activation while the other is held", () => {
+        const elA = createInput({value: "0", min: "0", max: "10", step: "1"})
+        const elB = createInput({value: "0", min: "0", max: "10", step: "1"})
+        new InputSpinner(elA, {autoInterval: undefined})
+        new InputSpinner(elB, {autoInterval: undefined})
+        const btnA = elA.nextElementSibling.querySelector(".btn-increment")
+        const btnB = elB.nextElementSibling.querySelector(".btn-increment")
+        const make = (type) => {
+            const e = new KeyboardEvent(type, {key: " ", bubbles: true})
+            Object.defineProperty(e, "keyCode", {value: 32})
+            return e
+        }
+        const down = () => make("keydown")
+        const up = () => make("keyup")
+        btnA.dispatchEvent(down())
+        assert.equal(elA.value, "1")
+        // B's keydown must still register even though A's keyup hasn't fired.
+        btnB.dispatchEvent(down())
+        assert.equal(elB.value, "1")
+        document.body.dispatchEvent(up())
+        clear()
+    })
 })
 
 describe("InputSpinner attribute observation", () => {