Skip to content

Performance: Improve homepage CLS score#6438

Merged
leecalcote merged 1 commit intomasterfrom
leecalcote/performance/static-sized-banner-containers
May 8, 2025
Merged

Performance: Improve homepage CLS score#6438
leecalcote merged 1 commit intomasterfrom
leecalcote/performance/static-sized-banner-containers

Conversation

@leecalcote
Copy link
Copy Markdown
Member

Description

Performance: Improve CLS score with explicit height and width to containers for banners and lazy-loaded components

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Rian Cteulp <rian.cteulp@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2025

@l5io
Copy link
Copy Markdown
Member

l5io commented May 8, 2025

🚀 Preview for commit 50e486e at: https://681cf4c1ff7cb30f477841cb--layer5.netlify.app

@ritzorama
Copy link
Copy Markdown
Contributor

Thanks for your help, @leecalcote. Here is a link to the current externally run performance score for the homepage - https://pagespeed.web.dev/analysis/https-layer5-io/po3z5bs9kd?form_factor=mobile

@ritzorama
Copy link
Copy Markdown
Contributor

And here is a very recent Lighthouse report for the homepage, too - https://storage.googleapis.com/lighthouse-infrastructure.appspot.com/reports/1746684869452-25379.report.html. Weird how the score is ten points different between these two. Both of them in mobile view, too.

Copy link
Copy Markdown
Contributor

@zihanKuang zihanKuang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes reduce layout shifts on my end.🤠

@leecalcote
Copy link
Copy Markdown
Member Author

The changes reduce layout shifts on my end.🤠

Fantastic! I'm hopeful for an improved score. 😄

@leecalcote
Copy link
Copy Markdown
Member Author

It seems like we might have gotten a 6 point jump on the overall performance score in Lighthouse for mobile - https://storage.googleapis.com/lighthouse-infrastructure.appspot.com/reports/1746729617433-3434.report.html

@leecalcote leecalcote merged commit c123e8b into master May 8, 2025
10 checks passed
@leecalcote leecalcote deleted the leecalcote/performance/static-sized-banner-containers branch May 8, 2025 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants