Skip to content

Commit 0d057ae

Browse files
authored
Merge pull request #1578 from github/koesie10/mrva-performance
Fix freezing of live results view
2 parents c90eede + ebba994 commit 0d057ae

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

extensions/ql-vscode/src/view/variant-analysis/VariantAnalysis.tsx

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ export function VariantAnalysis({
6565
const [repoResults, setRepoResults] = useState<VariantAnalysisScannedRepositoryResult[]>(initialRepoResults);
6666

6767
useEffect(() => {
68-
window.addEventListener('message', (evt: MessageEvent) => {
68+
const listener = (evt: MessageEvent) => {
6969
if (evt.origin === window.origin) {
7070
const msg: ToVariantAnalysisMessage = evt.data;
7171
if (msg.t === 'setVariantAnalysis') {
@@ -89,8 +89,13 @@ export function VariantAnalysis({
8989
const origin = evt.origin.replace(/\n|\r/g, '');
9090
console.error(`Invalid event origin ${origin}`);
9191
}
92-
});
93-
});
92+
};
93+
window.addEventListener('message', listener);
94+
95+
return () => {
96+
window.removeEventListener('message', listener);
97+
};
98+
}, []);
9499

95100
if (variantAnalysis?.actionsWorkflowRunId === undefined) {
96101
return <VariantAnalysisLoading />;

0 commit comments

Comments
 (0)