From ddae9d45e692408169cb2dbe267badcc55b91ab2 Mon Sep 17 00:00:00 2001 From: ACTCD <101378590+ACTCD@users.noreply.github.com> Date: Tue, 22 Jul 2025 16:43:03 +0000 Subject: [PATCH 1/3] fix: cancel unnecessary state checks Avoid the inability to activate item[0] in the settings state, for example. --- src/ext/extension-page/Components/Sidebar/Sidebar.svelte | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/ext/extension-page/Components/Sidebar/Sidebar.svelte b/src/ext/extension-page/Components/Sidebar/Sidebar.svelte index cc382adc..5ca2ff1d 100644 --- a/src/ext/extension-page/Components/Sidebar/Sidebar.svelte +++ b/src/ext/extension-page/Components/Sidebar/Sidebar.svelte @@ -81,8 +81,8 @@ } async function activate(item) { - // if not in ready state or the item is already active - if (!$v4state.includes("ready") || item.active) return; + // if item is empty or already active + if (!item || item.active) return; // check if there's a temp item and it's not the item to be activated // can occur when user clicks a non-temp item while a temp item exists From 8e6d9cd319a198d13d361db95bd7b3a45bbe050d Mon Sep 17 00:00:00 2001 From: ACTCD <101378590+ACTCD@users.noreply.github.com> Date: Tue, 22 Jul 2025 17:04:53 +0000 Subject: [PATCH 2/3] fix: reduce the z-index of splitter --- src/ext/extension-page/App.svelte | 4 ++-- src/ext/extension-page/Components/Sidebar/Sidebar.svelte | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/ext/extension-page/App.svelte b/src/ext/extension-page/App.svelte index 90d396b4..ecdbdf52 100644 --- a/src/ext/extension-page/App.svelte +++ b/src/ext/extension-page/App.svelte @@ -168,7 +168,7 @@ cursor: col-resize; position: fixed; inset: 0; - z-index: 200; + z-index: 20; } .splitter { @@ -182,7 +182,7 @@ left: calc(var(--sidebar-fixed-width) - var(--splitter-offset)); width: var(--splitter-width); height: 100svh; - z-index: 201; + z-index: 21; transition: background-color 0.2s ease-in-out; &:hover, diff --git a/src/ext/extension-page/Components/Sidebar/Sidebar.svelte b/src/ext/extension-page/Components/Sidebar/Sidebar.svelte index 5ca2ff1d..c5af6b55 100644 --- a/src/ext/extension-page/Components/Sidebar/Sidebar.svelte +++ b/src/ext/extension-page/Components/Sidebar/Sidebar.svelte @@ -228,7 +228,7 @@ title="Open settings" {disabled} /> - + From 6bcab180bcb05e35d730993fbaa596eafdd7248f Mon Sep 17 00:00:00 2001 From: ACTCD <101378590+ACTCD@users.noreply.github.com> Date: Tue, 22 Jul 2025 17:05:46 +0000 Subject: [PATCH 3/3] fix: prevent hide sidebar when no item --- src/ext/extension-page/App.svelte | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/ext/extension-page/App.svelte b/src/ext/extension-page/App.svelte index ecdbdf52..98106604 100644 --- a/src/ext/extension-page/App.svelte +++ b/src/ext/extension-page/App.svelte @@ -66,6 +66,10 @@ const editorMinWidth = "20rem"; function sidebarSwitch() { + if (!sidebarHidden && !$items.find((i) => i.active)) { + log.add("Please select or create a new item first.", "info", true); + return; + } sidebarHidden = !sidebarHidden; }