Skip to content

Commit 8199219

Browse files
committed
refactor: Use for event listener cleanup
- Replace window.addEventListener with pattern - Add proper cleanup on component unmount - Prevent memory leaks in Svelte 5
1 parent 6eb6339 commit 8199219

1 file changed

Lines changed: 9 additions & 2 deletions

File tree

src/lib/SideBars/Sidebar.svelte

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -256,12 +256,19 @@
256256
}, 100)
257257
}
258258
259-
if (typeof window !== 'undefined') {
259+
$effect(() => {
260+
if (typeof window === 'undefined') return
261+
260262
const handler = () => {
261263
scrollToActiveCharacter()
262264
}
265+
263266
window.addEventListener('scrollToActiveCharacter', handler)
264-
}
267+
268+
return () => {
269+
window.removeEventListener('scrollToActiveCharacter', handler)
270+
}
271+
})
265272
266273
267274
const createFolder = (mainIndex:DragData, targetIndex:DragData) => {

0 commit comments

Comments
 (0)