Merge pull request #3213 from jspsych/add-response-ends-trial-vs-circle

Fix display clearing bug in visual search circle plugin version 1.2.0
jspsych · Jan 14, 2024 · daea41b · daea41b
2 parents e278bff + 7b79772
commit daea41b
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 0 deletions.
diff --git a/.changeset/clever-coats-invent.md b/.changeset/clever-coats-invent.md
@@ -0,0 +1,5 @@
+---
+"@jspsych/plugin-visual-search-circle": patch
+---
+
+Fix display clearing problem introduced with version 1.2.0
diff --git a/packages/plugin-visual-search-circle/src/index.spec.ts b/packages/plugin-visual-search-circle/src/index.spec.ts
@@ -27,6 +27,8 @@ describe("visual-search-circle", () => {
     pressKey("a");
     await expectFinished();
 
+    expect(displayElement.querySelectorAll("img").length).toBe(0);
+
     expect(getData().values()[0].correct).toBe(true);
   });
 
@@ -57,6 +59,8 @@ describe("visual-search-circle", () => {
     jest.runAllTimers();
     await expectFinished();
 
+    expect(displayElement.querySelectorAll("img").length).toBe(0);
+
     expect(getData().values()[0].correct).toBe(true);
   });
 });

diff --git a/packages/plugin-visual-search-circle/src/index.ts b/packages/plugin-visual-search-circle/src/index.ts
@@ -166,6 +166,8 @@ class VisualSearchCirclePlugin implements JsPsychPlugin<Info> {
     };
 
     const end_trial = () => {
+      display_element.innerHTML = "";
+
       this.jsPsych.pluginAPI.clearAllTimeouts();
       this.jsPsych.pluginAPI.cancelAllKeyboardResponses();