Closed lakshaydewan closed 2 weeks ago
[!CAUTION]
Review failed
The pull request is closed.
The changes in this pull request involve modifications to the Next.js application, specifically in the way scripts and styles are managed. The <script>
tags for loading external JavaScript have been replaced with the <Script>
component from next/script
, utilizing the strategy='lazyOnload'
attribute for deferred loading. Additionally, corrections were made to the <Head>
section in the _document.tsx
file, including a fix for the Google Fonts link and the addition of a new link for preloading the Fira Code font. An import for the Fira Code font was removed from the global CSS file.
File | Change Summary |
---|---|
pages/_app.tsx | Replaced <script> tag with <Script> component from next/script with strategy='lazyOnload' . |
pages/_document.tsx | Added import for Script , corrected rel attribute for Google Fonts link, added preload link for Fira Code, replaced GitHub buttons script with <Script> . |
styles/globals.css | Removed import for Fira Code font while keeping other CSS imports intact. |
<Script>
component.ready-to-merge
, bounty
🐰 In the garden where scripts do play,
A lazy load brightens the day.
Fonts now preload with style so neat,
Performance blooms, a joyful feat!
With each change, our app takes flight,
Hopping high, oh what a sight! 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Built without sensitive environment variables
Name | Link |
---|---|
Latest commit | 1e2f9e4c856391027b4936dc984df00f531cfa2a |
Latest deploy log | https://app.netlify.com/sites/asyncapi-website/deploys/672ba61c9b79060008d2515f |
Deploy Preview | https://deploy-preview-3367--asyncapi-website.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
We require all PRs to follow Conventional Commits specification. More details 👇🏼
The subject "Improving the lighthouse Score#3356" found in the pull request title "fix: Improving the lighthouse Score#3356" should start with a lowercase character.
⚡️ Lighthouse report for the changes in this PR: | Category | Score |
---|---|---|
🔴 Performance | 47 | |
🟢 Accessibility | 98 | |
🟢 Best practices | 92 | |
🟢 SEO | 100 | |
🔴 PWA | 33 |
Lighthouse ran on https://deploy-preview-3367--asyncapi-website.netlify.app/
Improved the lighthouse score on mobile devices by preloading fonts and using Native Script tag from next/script
the FCP of the website went down from 3 seconds average to 1.5 seconds improving the performance greatly.
Summary by CodeRabbit
New Features
<Script>
component for improved performance.Bug Fixes
Chores