Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(hierarchical-menu): persist refined root level count with a future flag #5950

Merged
merged 6 commits into from
Dec 6, 2023

Conversation

dhayab
Copy link
Member

@dhayab dhayab commented Nov 30, 2023

Summary

The current design of hierarchical facets in InstantSearch is to show the count of the active refined parent level next to its root level. This is not necessarily what current users of InstantSearch expect, so this PR provides an option in the form a future flag, to allow them to switch to a new behaviour where the root level count is not modified in answer to refinement changes.

Result

Users can now choose whether they want a hierarchical facet parent count reflected in the root level, or leave the root count untouched.

FX-2671

Copy link

codesandbox-ci bot commented Nov 30, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit c9ce3e8:

Sandbox Source
InstantSearch.js Configuration
react-instantsearch-app Configuration
example-react-instantsearch-default-theme Configuration
example-react-instantsearch-next-app-dir-example Configuration
example-vue-instantsearch-default-theme Configuration

@dhayab dhayab marked this pull request as ready for review December 4, 2023 15:45
@dhayab dhayab requested review from a team, sarahdayan and aymeric-giraudet and removed request for a team December 4, 2023 15:45
@dhayab dhayab merged commit 79ee485 into master Dec 6, 2023
@dhayab dhayab deleted the feat/hierarchical-menu-persist-root-count branch December 6, 2023 10:50
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.

2 participants