From 9092bd41ade0f356d3109018ef177cf5fe60f802 Mon Sep 17 00:00:00 2001 From: LinoH5 Date: Wed, 9 Aug 2023 18:16:18 +0200 Subject: [PATCH 1/2] remove pictures from show-more for location and date, unfold TagOverview --- .../src/components/common/TagOverview.tsx | 3 + .../src/components/views/search/TagList.tsx | 5 +- .../views/show-more/ShowMoreView.tsx | 62 ++++++++++--------- .../views/show-more/ShowMoreViewHeader.tsx | 1 + 4 files changed, 40 insertions(+), 31 deletions(-) diff --git a/projects/bp-gallery/src/components/common/TagOverview.tsx b/projects/bp-gallery/src/components/common/TagOverview.tsx index 0a40c7ba6..c297a97c6 100644 --- a/projects/bp-gallery/src/components/common/TagOverview.tsx +++ b/projects/bp-gallery/src/components/common/TagOverview.tsx @@ -27,6 +27,7 @@ interface TagOverviewProps { queryParams?: LocationTagFiltersInput | PersonTagFiltersInput | KeywordTagFiltersInput; thumbnailQueryParams?: PictureFiltersInput; archiveId?: string; + allowFold?: boolean; } const TagOverview = ({ @@ -37,6 +38,7 @@ const TagOverview = ({ queryParams, thumbnailQueryParams, archiveId, + allowFold, }: TagOverviewProps) => { const { t } = useTranslation(); const ref = useRef(null); @@ -114,6 +116,7 @@ const TagOverview = ({ currentItemAmount={rows ? rowLength * rows : undefined} queryParams={queryParams} thumbnailQueryParams={thumbnailQueryParams} + allowFold={allowFold} /> ) : ( { const { visit, location } = useVisit(); const { t } = useTranslation(); @@ -73,7 +75,8 @@ const TagList = ({ ); const isFoldable = Boolean( - flattenedTags && + allowFold && + flattenedTags && elementsPerRow && flattenedTags.length > MAX_ROWS_WITHOUT_FOLDING * elementsPerRow && !currentItemAmount diff --git a/projects/bp-gallery/src/components/views/show-more/ShowMoreView.tsx b/projects/bp-gallery/src/components/views/show-more/ShowMoreView.tsx index 78dc63530..5cb552508 100644 --- a/projects/bp-gallery/src/components/views/show-more/ShowMoreView.tsx +++ b/projects/bp-gallery/src/components/views/show-more/ShowMoreView.tsx @@ -82,36 +82,38 @@ const ShowMoreView = ({ flattenedTags={flattenedTags} /> - + {((categoryType !== 'location' && categoryType !== 'date') || categoryId) && ( + + )} ); diff --git a/projects/bp-gallery/src/components/views/show-more/ShowMoreViewHeader.tsx b/projects/bp-gallery/src/components/views/show-more/ShowMoreViewHeader.tsx index 6b9aab089..e74e73859 100644 --- a/projects/bp-gallery/src/components/views/show-more/ShowMoreViewHeader.tsx +++ b/projects/bp-gallery/src/components/views/show-more/ShowMoreViewHeader.tsx @@ -76,6 +76,7 @@ const ShowMoreViewHeader = ({ type={categoryType as TagType} queryParams={getCategoryQueryParams(archiveId)} archiveId={archiveId} + allowFold={false} /> ); From 0c7cd3a703012d46be07ede4c5fe7be6a12447d9 Mon Sep 17 00:00:00 2001 From: LinoH5 Date: Wed, 9 Aug 2023 20:53:24 +0200 Subject: [PATCH 2/2] fix tests --- projects/bp-gallery/cypress/e2e/show-more.cy.ts | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/projects/bp-gallery/cypress/e2e/show-more.cy.ts b/projects/bp-gallery/cypress/e2e/show-more.cy.ts index 571262b1b..2020403db 100644 --- a/projects/bp-gallery/cypress/e2e/show-more.cy.ts +++ b/projects/bp-gallery/cypress/e2e/show-more.cy.ts @@ -148,12 +148,6 @@ describe('Global Show More View', () => { targetText ); }); - - // check for images in show more view - cy.contains('Mehr als 100 Bilder'); - for (const id of [2, 3, 4, 5]) { - cy.get(`.picture-grid .row #picture-preview-for-${id}`).should('exist'); - } }); it('shows show more view for single decade "70er"', () => { @@ -181,12 +175,6 @@ describe('Global Show More View', () => { `VERIFIZIERTER TESTORT ${num}` ); } - - // check for images in show more view - cy.contains('Mehr als 100 Bilder'); - for (const id of [2, 3, 4, 5]) { - cy.get(`.picture-grid .row #picture-preview-for-${id}`).should('exist'); - } }); it('shows show more for single location "Verifizierter Testort 3"', () => {