Skip to content

Conversation

@Azizah2023
Copy link
Contributor

@Azizah2023 Azizah2023 commented Nov 23, 2025

Description

Changes

  • Lazy-load non-critical sections on the Kanvas page using import() + useEffect.
  • Defer loading of non-critical Kanvas sections — Reviews, TeaserModal, KanvasFeatures, Catalog, KanvasBtn, and Platform — to reduce unused JavaScript and improve initial page performance.
  • Performance before
Screenshot 1447-06-01 at 11 54 01 PM
  • Performance after
Screenshot 1447-06-02 at 1 22 28 AM

Notes

  • This is a clean replacement for previous work on branch fix/seo-alt-links-kanvas, which became noisy after syncing with master.

This PR fixes #
#7110

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@l5io
Copy link
Contributor

l5io commented Nov 23, 2025

🚀 Preview for commit 96d1cc0 at: https://6922e1d23100b6619275b4a6--layer5.netlify.app

@saurabhraghuvanshii
Copy link
Contributor

@Azizah2023 Please remove all unnecessary lint changes such as extra lines and spaces, and keep only the original logic.

@l5io
Copy link
Contributor

l5io commented Nov 23, 2025

🚀 Preview for commit dbaf1ba at: https://6922f3322f7dd379d9da1bc4--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Nov 25, 2025

🚀 Preview for commit c0877e6 at: https://692591ed3e676a76d57e9e24--layer5.netlify.app

@kishore08-07
Copy link
Contributor

kishore08-07 commented Nov 28, 2025

Checks are currently failing, possibly due to the latest Gatsby update
Other than that, the changes look good to me @Azizah2023

…cript

Signed-off-by: Azizah2023 <A.alshehri@thegarage.sa>
Signed-off-by: Azizah2023 <A.alshehri@thegarage.sa>
@Azizah2023 Azizah2023 force-pushed the perf/kanvas-lazy-load branch from e348de9 to 014680e Compare November 30, 2025 11:02
@github-actions github-actions bot added area/landscape The service mesh landscape area/projects An issue relating to Layer5 initiatives (projects) area/community area/learn Related to /learn section area/handbook project/sistent labels Nov 30, 2025
Signed-off-by: Azizah <121037475+Azizah2023@users.noreply.github.com>
@kishore08-07
Copy link
Contributor

@Azizah2023
kindly resolve the merge conflicts

@leecalcote leecalcote requested a review from Fireentity December 1, 2025 07:15
Azizah2023 and others added 3 commits December 1, 2025 11:03
Signed-off-by: Azizah2023 <A.alshehri@thegarage.sa>
Signed-off-by: Azizah <121037475+Azizah2023@users.noreply.github.com>
Copy link
Contributor

@Fireentity Fireentity left a comment

Choose a reason for hiding this comment

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

@Azizah2023
Please ensure that your commit messages are meaningful rather than repeating the same message, as we need a clear history of changes. Finally, I noticed some files only contain formatting updates (like spacing). Please revert those specific modifications so we can keep the pull request focused strictly on functional code changes.

@Fireentity
Copy link
Contributor

@Azizah2023 Thank you for your contribution! Let's discuss this during the website call today at 6:30 PM IST | 8 AM CT Add it as an agenda item to the meeting minutes, if you would 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/community area/handbook area/landscape The service mesh landscape area/learn Related to /learn section area/projects An issue relating to Layer5 initiatives (projects) project/kanvas project/sistent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants