Skip to content

feat(filterSuggestions): add minimum loading duration to prevent skeleton flash#6878

Merged
FabienMotte merged 3 commits intomasterfrom
feat/filter-suggestions-min-loading-duration
Jan 27, 2026
Merged

feat(filterSuggestions): add minimum loading duration to prevent skeleton flash#6878
FabienMotte merged 3 commits intomasterfrom
feat/filter-suggestions-min-loading-duration

Conversation

@FabienMotte
Copy link
Contributor

Summary

This PR adds a minimum 300ms loading duration for the filter suggestions skeleton to prevent a flash when results are returned quickly from the Agent Studio API cache.

Result

When cached results return in <300ms, the skeleton now displays for the full 300ms before showing results, providing a smoother visual transition.

@FabienMotte FabienMotte self-assigned this Jan 27, 2026
Copy link
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

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

looks good. Should we add a test for this?

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 27, 2026

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 91a965e:

Sandbox Source
example-instantsearch-getting-started Configuration
example-react-instantsearch-getting-started Configuration
example-react-instantsearch-next-app-dir-example Configuration
example-react-instantsearch-next-routing-example Configuration
example-vue-instantsearch-getting-started Configuration

@FabienMotte
Copy link
Contributor Author

looks good. Should we add a test for this?

I fixed the existing tests and added a new one

@shaejaz shaejaz mentioned this pull request Jan 27, 2026
@FabienMotte FabienMotte merged commit ed5b86d into master Jan 27, 2026
14 checks passed
@FabienMotte FabienMotte deleted the feat/filter-suggestions-min-loading-duration branch January 27, 2026 15:54
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