-
Notifications
You must be signed in to change notification settings - Fork 1
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
Prefetch occurrences of the week #103
Comments
I think the better solution would be to use a proper loading spinner in this case. |
Just to clarify, these fast swaps to "Loading..." and back to content are the ones I think we should avoid: Screen.Recording.2022-07-24.at.16.55.57.mov |
Ghost elements, as they are called, are a solid option for initial page load. After initial load is done that we should definitely pre-fetch the other days in terms of user experience. The backend should cache those kinds requests anyway so the impact shouldn't be that significant. |
It sould be quite easy to implement prefetching: https://www.apollographql.com/docs/react/performance/performance/#prefetching-data |
With #93 being fixed, there is still a small amount of flicker due to the backend's latency.
Could we preload all occurrences of the currently select week (after page load) to smooth out the experience?
This should not interfere with normal site usage, as the data is being downloaded in the background.
There are valid data usage concerns, but I think the quality of life improvement is more important.
The text was updated successfully, but these errors were encountered: