8000 fix(loading): Prevent isLoading from being true with fallbackData by Conqxeror · Pull Request #4140 · vercel/swr · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix(loading): Prevent isLoading from being true with fallbackData #4140

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Conqxeror
Copy link

This PR addresses an issue where i 8000 sLoading was incorrectly set to true even when fallbackData was provided, particularly in SSR scenarios. This led to a "flash of skeleton" UI, which is undesirable when initial data is already available.

The change modifies the logic to ensure that isLoading remains false when fallbackData is present, while isValidating correctly reflects the background revalidation process. This allows the initial server-cached data to be displayed immediately, providing a smoother user experience.

Fixes #3046

When `fallbackData` is provided, `isLoading` should not be true initially.
Instead, `isValidating` should be true to indicate background revalidation.
This prevents a "flash of skeleton" UI when initial data is available.

Fixes vercel#3046
When `fallbackData` is provided, `isLoading` should not be true initially.
Instead, `isValidating` should be true to indicate background revalidation.
This prevents a "flash of skeleton" UI when initial data is available.

Fixes vercel#3046
@Conqxeror Conqxeror requested review from shuding and huozhi as code owners June 28, 2025 23:28
Copy link

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.

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.

revalidateOnMount With Fallback triggers isLoading instead of isValidating
1 participant
0