diff --git a/frontend/src/components/layout/SidebarComponent.test.tsx b/frontend/src/components/layout/SidebarComponent.test.tsx index 8c7879a..3316c91 100644 --- a/frontend/src/components/layout/SidebarComponent.test.tsx +++ b/frontend/src/components/layout/SidebarComponent.test.tsx @@ -70,6 +70,16 @@ describe ('SidebarComponent', () => { vi.spyOn (HTMLElement.prototype, 'hasPointerCapture').mockReturnValue (true) }) + it ('limits sidebar scrolling to desktop to avoid clipping mobile suggestions', () => { + renderSidebar () + + // jsdom has no layout; guard the responsive classes responsible for clipping. + const sidebar = screen.getByText ('Sidebar body') + + expect (sidebar).not.toHaveClass ('overflow-y-auto') + expect (sidebar).toHaveClass ('md:overflow-y-auto') + }) + it ('uses maxWidth without applying the container main-width reserve twice', async () => { localStorage.setItem ('sidebar:test-sidebar:left:width', '700') const onWidthChange = vi.fn () diff --git a/frontend/src/components/layout/SidebarComponent.tsx b/frontend/src/components/layout/SidebarComponent.tsx index 6e2b29c..5157c78 100644 --- a/frontend/src/components/layout/SidebarComponent.tsx +++ b/frontend/src/components/layout/SidebarComponent.tsx @@ -339,7 +339,7 @@ const SidebarComponent: FC = ({ -
+
{children}
diff --git a/frontend/src/pages/tags/TagDetailPage.test.tsx b/frontend/src/pages/tags/TagDetailPage.test.tsx index 567c554..e86cbbb 100644 --- a/frontend/src/pages/tags/TagDetailPage.test.tsx +++ b/frontend/src/pages/tags/TagDetailPage.test.tsx @@ -35,6 +35,10 @@ const renderPage = () => describe ('TagDetailPage', () => { beforeEach (() => { vi.clearAllMocks () + tagsApi.fetchTag.mockReset () + api.apiPut.mockReset () + api.isApiError.mockReset () + toastApi.toast.mockReset () api.isApiError.mockReturnValue (false) }) @@ -51,8 +55,11 @@ describe ('TagDetailPage', () => { }) it ('submits edited tag fields', async () => { - tagsApi.fetchTag.mockResolvedValueOnce (buildTag ({ id: 7, name: 'old' })) - api.apiPut.mockResolvedValueOnce (buildTag ({ id: 7, name: 'new', aliases: ['alias'] })) + const updatedTag = buildTag ({ id: 7, name: 'new', aliases: ['alias'] }) + tagsApi.fetchTag + .mockResolvedValue (updatedTag) + .mockResolvedValueOnce (buildTag ({ id: 7, name: 'old' })) + api.apiPut.mockResolvedValueOnce (updatedTag) renderPage ()