diff --git a/src/renderer/src/components/sidebar/SidebarNav.test.tsx b/src/renderer/src/components/sidebar/SidebarNav.test.tsx index 87b51ee5b23..cf4f17295b0 100644 --- a/src/renderer/src/components/sidebar/SidebarNav.test.tsx +++ b/src/renderer/src/components/sidebar/SidebarNav.test.tsx @@ -429,6 +429,20 @@ describe('SidebarNav', () => { expect(mocks.updateSettings).toHaveBeenCalledWith({ showMobileButton: false }) }) + it('places the worktree palette search above the sidebar nav rows', async () => { + const container = await renderSidebarNav() + const nav = container.querySelector('[data-contextual-tour-target="sidebar-navigation"]') + const searchButton = container.querySelector( + 'button[aria-label="Search worktrees and browser tabs"]' + ) + const tasksButton = getButtonByText(container, 'Tasks') + + expect(nav?.firstElementChild).toBe(searchButton) + expect( + searchButton?.compareDocumentPosition(tasksButton) & Node.DOCUMENT_POSITION_FOLLOWING + ).toBeTruthy() + }) + it('hides the worktree palette shortcut until the search field is hovered or focused', async () => { const container = await renderSidebarNav() @@ -439,8 +453,8 @@ describe('SidebarNav', () => { const shortcuts = searchButton?.querySelector('span.hidden') expect(shortcuts?.className).toContain('hidden') - expect(shortcuts?.className).toContain('group-hover:inline-flex') - expect(shortcuts?.className).toContain('group-focus-within:inline-flex') + expect(shortcuts?.className).toContain('group-hover:flex') + expect(shortcuts?.className).toContain('group-focus-within:flex') expect(shortcuts?.textContent).toContain('⌘') expect(shortcuts?.textContent).toContain('J') expect(searchButton?.querySelector('kbd')).toBeNull() diff --git a/src/renderer/src/components/sidebar/SidebarNav.tsx b/src/renderer/src/components/sidebar/SidebarNav.tsx index 3133f7aa6d1..c5827db332d 100644 --- a/src/renderer/src/components/sidebar/SidebarNav.tsx +++ b/src/renderer/src/components/sidebar/SidebarNav.tsx @@ -106,6 +106,35 @@ const SidebarNav = React.memo(function SidebarNav() { className="flex flex-col gap-0.5 px-2 pt-2 pb-1" data-contextual-tour-target="sidebar-navigation" > + {showArtifactsButton ? ( @@ -302,35 +331,6 @@ const SidebarNav = React.memo(function SidebarNav() { ) : null} - ) })