Skip to content

Commit f383f1f

Browse files
authored
fix: do not update coverage in loop for faster render (#23)
1 parent f5780ac commit f383f1f

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/routes/(public)/css-coverage/+page.svelte

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
3838
async function onchange(event: Event) {
3939
let files = (event.target as HTMLInputElement)?.files
40-
data = []
40+
let new_data: Coverage[] = []
4141
4242
if (!files) return
4343
@@ -48,8 +48,11 @@
4848
}
4949
let text = await file.text()
5050
let parsed = parse_json(text)
51-
data = [...data, ...parsed]
51+
new_data.push(...parsed)
5252
}
53+
54+
// only update state once to prevent hundreds of re-renders
55+
data = new_data
5356
}
5457
5558
let drag_state: 'idle' | 'dragging' = $state('idle')

0 commit comments

Comments
 (0)