Merge pull request #3563 from farhat60/bugfix/3562-image-button-enable_after

fix: enable_button_after parameter in plugin-image-button-response (closes #3562)

Author: jodeleeuw

.changeset/busy-tires-reply.md

@@ -0,0 +1,5 @@
+---
+"@jspsych/plugin-image-button-response": minor
+---
+
+CSS selector that queries the button to disable it was trying to get a class. Now it looks for the correct id and disables the button.

packages/plugin-image-button-response/src/index.spec.ts

@@ -147,7 +147,7 @@ describe("image-button-response", () => {
       },
     ]);
 
-    const btns = displayElement.querySelectorAll(".jspsych-image-button-response-button button");
+    const btns = displayElement.querySelectorAll("#jspsych-image-button-response-btngroup button");
 
     for (let i = 0; i < btns.length; i++) {
       expect(btns[i].getAttribute("disabled")).toBe("disabled");

packages/plugin-image-button-response/src/index.ts

@@ -306,14 +306,14 @@ class ImageButtonResponsePlugin implements JsPsychPlugin<Info> {
     }
 
     function enable_buttons() {
-      var btns = document.querySelectorAll(".jspsych-image-button-response-button button");
+      var btns = document.querySelectorAll("#jspsych-image-button-response-btngroup button");
       for (var i = 0; i < btns.length; i++) {
         btns[i].removeAttribute("disabled");
       }
     }
 
     function disable_buttons() {
-      var btns = document.querySelectorAll(".jspsych-image-button-response-button button");
+      var btns = document.querySelectorAll("#jspsych-image-button-response-btngroup button");
       for (var i = 0; i < btns.length; i++) {
         btns[i].setAttribute("disabled", "disabled");
       }