-
-
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
Homepage code and image optimization #87
Comments
|
There are potentially two image lazy-loaders in effect here: Using http://mstoday.staging.wpengine.com/2020/06/15/ihl-college-board-to-approve-relocating-confederate-monument-from-heart-of-ole-miss-campus/ as an example, here's Chrome's Lighthouse report: And Safari's "Timelines" audit, which isn't anywhere near as featureful as Chrome: Steps taken:
Afterwards, Chrome and Safari: Nevermind, then. This looks like a performance detriment. |
Twitter's own widget generator does not offer this option, at https://publish.twitter.com/?dnt=1&query=https%3A%2F%2Ftwitter.com%2FMSTODAYnews&widget=Timeline Options:
|
Can we do some quick research with the remaining time to see if there's a plugin that would work for this? |
@MirandaEcho A few options for Twitter:
|
Great, let's give that first one a shot on staging. We can wait until we do the other deploys first though, so:
|
@MirandaEcho We'll need to ask Lauchlin to provide us with these 4 keys in order to configure the Twitter plugin:
|
@joshdarby Lauchlin wasn't able to find those in her twitter account, but she sent the creds and they're in 1Pass so we can grab the needed keys ourselves. |
@MirandaEcho The twitter widget is now enabled at http://mstoday.staging.wpengine.com/ |
Thanks @joshdarby! Are we able to tell if it does better performance-wise? |
@MirandaEcho The prod site has a score of 2/100 while the staging site now has a score of 22/100. However, staging doesn't have all of the caching and whatnot that prod does, so it's kind of hard to get a full scope of what the performance increase will be. However, lighthouse isn't complaining about the new twitter widget, which is a good sign. |
No description provided.
The text was updated successfully, but these errors were encountered: