Fix selection created on videos page refresh

probablykasper · Jan 28, 2024 · 3f8ed87 · 3f8ed87
1 parent f76926b
commit 3f8ed87
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,8 @@
 # Changelog
 
+## Next
+- Fix selection being created on videos page refresh
+
 ## 1.7.0 - 2024 Jan 21
 - Add basic history page, accessible via `Cmd/Ctrl+Y`.
 - Fix selection staying in the same position when videos update

diff --git a/src/routes/+page.svelte b/src/routes/+page.svelte
@@ -24,15 +24,14 @@
 
 		// Update the selection index if the video moves
 		const newSelectedIndex = videos.findIndex((v) => v.id === selectedId)
-		if (newSelectedIndex >= 0) {
+		if (newSelectedIndex >= 0 && selectionVisible) {
 			selectedIndex = newSelectedIndex
-			selectionVisible = true
 		} else {
 			// Or clear selection if the video disappeared from view
 			selectedIndex = 0
 			selectionVisible = false
 		}
-		const selectionMoved = selectionVisible && selectedIndex !== oldselectedIndex
+		const selectionMoved = selectedIndex !== oldselectedIndex && selectionVisible
 
 		if (selectionMoved) {
 			allowScrollToBox = false