-
Notifications
You must be signed in to change notification settings - Fork 498
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
Improve load-time performance of widgets #6008
Labels
Comments
Exciting @Haroenv - I think https://sorare.com/football/market/new-signings could be a great production use-case to confirm those numbers ; I can give this a try next week |
@Haroenv I'm a bit confused, is instantsearch.js |
Hi @redox, Haroen is off so I'll answer in his place. Today's release of [email protected], [email protected] and [email protected] include the PRs mentioned in this issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When mounting a lot of widgets, the time to mount can add up quite a bit. This issue highlights a couple ways to deal with it. These PRs are ordered in the ordering they should end up in
master
.Times are for an extreme app which mounts hundreds of widgets using DynamicWidgets.
The applied changed in 🟢 are available in InstantSearch.js 4.64.1, React InstantSearch 7.5.3, Vue InstantSearch 4.13.5
swap clearRefinements for individual refinement clearing methods which are more efficient 🟢
Fix sort of facet values in requestBuilder 🟢
avoid a copy of parameters to set in
setQueryParameters
andnew SearchParameters
🟢Don't recompute search parameters for every widget 🔴
instantsearch/packages/instantsearch.js/src/widgets/index/index.ts
Lines 326 to 333 in 2258d89
Don't clean up "unused" UiState and parameters 🔴
The text was updated successfully, but these errors were encountered: