Skip to content

Commit daea41b

Browse files
authored
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
2 parents e278bff + 7b79772 commit daea41b

File tree

3 files changed

+11
-0
lines changed

3 files changed

+11
-0
lines changed

.changeset/clever-coats-invent.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@jspsych/plugin-visual-search-circle": patch
3+
---
4+
5+
Fix display clearing problem introduced with version 1.2.0

packages/plugin-visual-search-circle/src/index.spec.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,8 @@ describe("visual-search-circle", () => {
2727
pressKey("a");
2828
await expectFinished();
2929

30+
expect(displayElement.querySelectorAll("img").length).toBe(0);
31+
3032
expect(getData().values()[0].correct).toBe(true);
3133
});
3234

@@ -57,6 +59,8 @@ describe("visual-search-circle", () => {
5759
jest.runAllTimers();
5860
await expectFinished();
5961

62+
expect(displayElement.querySelectorAll("img").length).toBe(0);
63+
6064
expect(getData().values()[0].correct).toBe(true);
6165
});
6266
});

packages/plugin-visual-search-circle/src/index.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,8 @@ class VisualSearchCirclePlugin implements JsPsychPlugin<Info> {
166166
};
167167

168168
const end_trial = () => {
169+
display_element.innerHTML = "";
170+
169171
this.jsPsych.pluginAPI.clearAllTimeouts();
170172
this.jsPsych.pluginAPI.cancelAllKeyboardResponses();
171173

0 commit comments

Comments
 (0)