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

Partial prerendering of public pages for better core web vitals #740

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

benmccann
Copy link
Contributor

@benmccann benmccann commented Jan 23, 2024

Closes #722

Previously it would display a blank white screen, followed by a loading widget, followed by the actual content.

Now it displays the left half of the content with the logo and title immediately and then the right half of the content with page-specific contents is filled in later.

Note that turning on SSR would be a more performant solution for the rest of the pages and not just the public facing ones I'm tweaking here. However, I understand there's a desire to run only a single process (PHP server) and avoid the operational overhead of introducing a Node server, so I've sent this PR as a solution instead.

Copy link

vercel bot commented Jan 23, 2024

@benmccann is attempting to deploy a commit to the appwrite Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Jan 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
console-cloud ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 30, 2024 2:56pm
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant