From b047552d7d7c72794dfe6719379d54f16dc9942f Mon Sep 17 00:00:00 2001 From: Justin Edmund Date: Wed, 3 Dec 2025 17:57:50 -0800 Subject: [PATCH] fix infinite loop: use onMount instead of $effect for sidebar action --- .../components/collection/CollectionArtifactDetailPane.svelte | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/lib/components/collection/CollectionArtifactDetailPane.svelte b/src/lib/components/collection/CollectionArtifactDetailPane.svelte index 034d2220..eddd9406 100644 --- a/src/lib/components/collection/CollectionArtifactDetailPane.svelte +++ b/src/lib/components/collection/CollectionArtifactDetailPane.svelte @@ -7,6 +7,7 @@ * Shows artifact info in read-only mode. If user is owner, * provides Edit button in header to push edit pane onto stack. */ + import { onMount } from 'svelte' import type { CollectionArtifact } from '$lib/types/api/artifact' import { isQuirkArtifact } from '$lib/types/api/artifact' import { usePaneStack, type PaneConfig } from '$lib/stores/paneStack.svelte' @@ -64,7 +65,7 @@ } // Set up the Edit action button in the pane header for owners - $effect(() => { + onMount(() => { if (isOwner) { sidebar.setAction(handleEdit, 'Edit') }