From 03450bd6f4411fe4855583b89eb50626d4655ead Mon Sep 17 00:00:00 2001 From: Alexander Petric Date: Fri, 10 Jan 2025 13:41:50 -0500 Subject: [PATCH] ws color: separate logic from reactivity condition (#5046) * fix workspace color picker issue * removing useless reactive statement * some ui fixes * separate logic from reactivity condition --- .../src/lib/components/settings/ChangeWorkspaceColor.svelte | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/frontend/src/lib/components/settings/ChangeWorkspaceColor.svelte b/frontend/src/lib/components/settings/ChangeWorkspaceColor.svelte index bf5a85120e..1b79adad28 100644 --- a/frontend/src/lib/components/settings/ChangeWorkspaceColor.svelte +++ b/frontend/src/lib/components/settings/ChangeWorkspaceColor.svelte @@ -14,9 +14,11 @@ export let open = false - $: if ($usersWorkspaceStore && $workspaceStore !== lastWorkspace) { + $: $usersWorkspaceStore && $workspaceStore !== lastWorkspace && onWorkspaceChange() + + function onWorkspaceChange() { lastWorkspace = $workspaceStore - savedWorkspaceColor = $usersWorkspaceStore.workspaces.find( + savedWorkspaceColor = $usersWorkspaceStore?.workspaces.find( (w) => w.id === $workspaceStore )?.color workspaceColor = savedWorkspaceColor