Skip to content

Conversation

@rubenmoya
Copy link
Contributor

@rubenmoya rubenmoya commented Oct 21, 2025

Description

In Engagement, we need a new navigation filter to move through the different cycles for each survey.

image

Keep in mind, that even if you see dates there, we're not selecting date ranges from a calendar by a given granularity like we do in the date navigation. In this case is a list of cycles, and the granularity could be different, or non-existen if the survey is sent manually.

More information in this slack thread

Screenshots (if applicable)

Closed:
image

Open:
image

With different list:
image

Use in the DataCollection:
image

Implementation details

This PR implements a new F0ListNavigation component that allows users to navigate through a given list using arrows. This component is not exported, right now it's only used internally for the DataCollection.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 21, 2025

🔍 Visual review for your branch is published 🔍

Here are the links to:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2025

Coverage Report for packages/react

Status Category Percentage Covered / Total
🔵 Lines 17.8% 18368 / 103149
🔵 Statements 17.8% 18368 / 103149
🔵 Functions 43.64% 868 / 1989
🔵 Branches 70.29% 2603 / 3703
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/react/src/components/F0ListNavigation/F0ListNavigation.stories.tsx 0% 100% 100% 0% 2-50
packages/react/src/components/F0ListNavigation/F0ListNavigation.tsx 12.5% 100% 0% 12.5% 23-95
packages/react/src/components/F0ListNavigation/index.tsx 100% 100% 100% 100%
packages/react/src/experimental/OneDataCollection/__stories__/mockData.tsx 0% 100% 100% 0% 2-1515
packages/react/src/experimental/OneDataCollection/__stories__/navigation-filters.stories.tsx 0% 0% 0% 0% 1-172
packages/react/src/experimental/OneDataCollection/components/NavigationFilters/NavigationFilters.tsx 9.37% 100% 0% 9.37% 20-54
packages/react/src/experimental/OneDataCollection/hooks/useDataCollectionSource/useDataCollectionSource.ts 64% 75% 100% 64% 72-91
packages/react/src/experimental/OneDataCollection/navigationFilters/index.tsx 69.23% 100% 0% 69.23% 20-23
packages/react/src/experimental/OneDataCollection/navigationFilters/types.ts 100% 100% 100% 100%
packages/react/src/experimental/OneDataCollection/navigationFilters/filterTypes/DateNavigation/index.tsx 27.27% 100% 0% 27.27% 7-10, 18-39, 41-42
packages/react/src/experimental/OneDataCollection/navigationFilters/filterTypes/DateNavigation/types.ts 100% 100% 100% 100%
packages/react/src/experimental/OneDataCollection/navigationFilters/filterTypes/ListNavigation/ListNavigation.tsx 11.11% 100% 0% 11.11% 5-22
packages/react/src/experimental/OneDataCollection/navigationFilters/filterTypes/ListNavigation/index.tsx 100% 100% 0% 100%
packages/react/src/experimental/OneDataCollection/navigationFilters/filterTypes/ListNavigation/types.ts 100% 100% 100% 100%
Generated in workflow #8866 for commit 614c29d by the Vitest Coverage Report Action

@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2025

size-limit report 📦

Path Size
JS: Stable 601.73 KB (+0.08% 🔺)
JS: Experimental 973.68 KB (+0.02% 🔺)
CSS 71.73 KB (0%)

@rubenmoya rubenmoya force-pushed the add-navigation-list branch from 02766ea to d41245b Compare October 27, 2025 09:16
@rubenmoya rubenmoya force-pushed the add-navigation-list branch from dd1bc54 to 0d1bf5d Compare October 28, 2025 09:28
@rubenmoya rubenmoya marked this pull request as ready for review October 28, 2025 09:29
@rubenmoya rubenmoya requested a review from a team as a code owner October 28, 2025 09:29
@rubenmoya rubenmoya force-pushed the add-navigation-list branch from 00a31cb to 614c29d Compare October 28, 2025 11:43
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file was duplicated, we had a .ts one and a .tsx one. I've kept the one with the most recent changes.

@sergiocarracedo
Copy link
Collaborator

Does this PR makes sense now? If not, could you close it? if yes can you merge it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants