fix: fix memory leaking of Notes.svelte (#21963)
This commit is contained in:
@@ -288,7 +288,7 @@
|
|||||||
dragged = false;
|
dragged = false;
|
||||||
};
|
};
|
||||||
|
|
||||||
onMount(async () => {
|
onMount(() => {
|
||||||
viewOption = localStorage?.noteViewOption ?? null;
|
viewOption = localStorage?.noteViewOption ?? null;
|
||||||
displayOption = localStorage?.noteDisplayOption ?? null;
|
displayOption = localStorage?.noteDisplayOption ?? null;
|
||||||
|
|
||||||
@@ -298,18 +298,16 @@
|
|||||||
dropzoneElement?.addEventListener('dragover', onDragOver);
|
dropzoneElement?.addEventListener('dragover', onDragOver);
|
||||||
dropzoneElement?.addEventListener('drop', onDrop);
|
dropzoneElement?.addEventListener('drop', onDrop);
|
||||||
dropzoneElement?.addEventListener('dragleave', onDragLeave);
|
dropzoneElement?.addEventListener('dragleave', onDragLeave);
|
||||||
});
|
|
||||||
|
|
||||||
onDestroy(() => {
|
return () => {
|
||||||
clearTimeout(searchDebounceTimer);
|
clearTimeout(searchDebounceTimer);
|
||||||
console.log('destroy');
|
|
||||||
const dropzoneElement = document.getElementById('notes-container');
|
|
||||||
|
|
||||||
if (dropzoneElement) {
|
if (dropzoneElement) {
|
||||||
dropzoneElement?.removeEventListener('dragover', onDragOver);
|
dropzoneElement?.removeEventListener('dragover', onDragOver);
|
||||||
dropzoneElement?.removeEventListener('drop', onDrop);
|
dropzoneElement?.removeEventListener('drop', onDrop);
|
||||||
dropzoneElement?.removeEventListener('dragleave', onDragLeave);
|
dropzoneElement?.removeEventListener('dragleave', onDragLeave);
|
||||||
}
|
}
|
||||||
|
};
|
||||||
});
|
});
|
||||||
</script>
|
</script>
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user