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

Preload test #262

Merged
merged 3 commits into from
Nov 10, 2018
Merged

Preload test #262

merged 3 commits into from
Nov 10, 2018

Conversation

jakearchibald
Copy link
Collaborator

Testing what's faster

@jakearchibald
Copy link
Collaborator Author

Master: https://www.webpagetest.org/video/compare.php?tests=181110_CD_cab5f0e0903d97424b083d64fdfdae9f-r:1-c:0

Something's blocking first render and interaction. Looks like it's either CSS or analytics.

@jakearchibald
Copy link
Collaborator Author

Oh shit wait it's analytics. It's a blocking script in the head.

@jakearchibald
Copy link
Collaborator Author

The prerenderer is fucking up here. We add analytics dynamically, and it's keeping that dom for the prerender. I'll fix.

@jakearchibald
Copy link
Collaborator Author

(I mean, the prerenderer isn't fucking up, it's doing exactly what we asked it to do, we just didn't think it through)

@jakearchibald
Copy link
Collaborator Author

@jakearchibald
Copy link
Collaborator Author

Forgive me for merging this without review.

@jakearchibald jakearchibald merged commit a1a00f0 into master Nov 10, 2018
@jakearchibald jakearchibald deleted the preload-css branch November 10, 2018 16:20
alisaitbilgi pushed a commit to alisaitbilgi/squoosh that referenced this pull request Feb 19, 2019
* Preload test

* Don't prerender analytics

* Version bump
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.

1 participant