diff --git a/CHANGELOG.md b/CHANGELOG.md index e007e56a1..f43e7106f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -14,6 +14,7 @@ and this project adheres to ### Changed - ♻️(backend) do not paginate threads list response #2186 +- 💄(frontend) Use StyledLink for sub doc tree #2188 ### Fixed diff --git a/src/frontend/apps/e2e/__tests__/app-impress/doc-tree.spec.ts b/src/frontend/apps/e2e/__tests__/app-impress/doc-tree.spec.ts index a383c1522..3fe255c5b 100644 --- a/src/frontend/apps/e2e/__tests__/app-impress/doc-tree.spec.ts +++ b/src/frontend/apps/e2e/__tests__/app-impress/doc-tree.spec.ts @@ -135,7 +135,7 @@ test.describe('Doc Tree', () => { await expect(docTree).toBeVisible(); await docTree.getByText('keyboard_arrow_right').click(); await docTree - .getByRole('button', { + .getByRole('link', { name: `Open document ${titleChild}`, }) .click(); diff --git a/src/frontend/apps/impress/src/features/docs/doc-tree/components/DocSubPageItem.tsx b/src/frontend/apps/impress/src/features/docs/doc-tree/components/DocSubPageItem.tsx index 8f13ffee8..4345834e2 100644 --- a/src/frontend/apps/impress/src/features/docs/doc-tree/components/DocSubPageItem.tsx +++ b/src/frontend/apps/impress/src/features/docs/doc-tree/components/DocSubPageItem.tsx @@ -12,7 +12,7 @@ import { useEffect, useRef, useState } from 'react'; import { useTranslation } from 'react-i18next'; import { css } from 'styled-components'; -import { Box, BoxButton, Icon, Text } from '@/components'; +import { Box, Icon, StyledLink, Text } from '@/components'; import { useCunninghamTheme } from '@/cunningham'; import { Doc, @@ -283,17 +283,12 @@ const DocSubPageItemContent = (props: TreeViewNodeProps) => { buttonOptionRef={buttonOptionRef} /> - { - e.stopPropagation(); - handleActivate(); + { + treeContext?.treeData.setSelectedNode(doc); }} tabIndex={-1} - $width="100%" - $direction="row" - $gap={spacingsTokens['xs']} - $align="center" - $minHeight="24px" data-testid={`doc-sub-page-item-${doc.id}`} aria-label={`${t('Open document {{title}}', { title: docTitle })}`} $css={css` @@ -304,6 +299,8 @@ const DocSubPageItemContent = (props: TreeViewNodeProps) => { ) => { /> )} - + );