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

@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
@kishore08-07
Copy link
Contributor

@Azizah2023
kindly resolve the merge conflicts

@leecalcote leecalcote requested a review from Fireentity December 1, 2025 07:15
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 🙂

@Fireentity
Copy link
Contributor

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

@Azizah2023 Azizah2023 force-pushed the perf/kanvas-lazy-load branch from 7a93a83 to a5ed310 Compare December 8, 2025 09:51
@l5io
Copy link
Contributor

l5io commented Dec 8, 2025

🚀 Preview for commit a5ed310 at: https://6936a373e62416d0c58ccb6b--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Dec 8, 2025

🚀 Preview for commit 0e6d3de at: https://6936a9d70775ef02ce4727d2--layer5.netlify.app

@lekaf974 lekaf974 added the issue/dco Commit sign-off instructions label Dec 8, 2025
@github-actions
Copy link

github-actions bot commented Dec 8, 2025

🚨 Alert! Git Police! We couldn’t help but notice that one or more of your commits is missing a sign-off. A what? A commit sign-off (your email address).

To amend the commits in this PR with your signoff using the instructions provided in the DCO check.

To configure your dev environment to automatically signoff on your commits in the future, see these instructions.


        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

1 similar comment
@github-actions
Copy link

github-actions bot commented Dec 8, 2025

🚨 Alert! Git Police! We couldn’t help but notice that one or more of your commits is missing a sign-off. A what? A commit sign-off (your email address).

To amend the commits in this PR with your signoff using the instructions provided in the DCO check.

To configure your dev environment to automatically signoff on your commits in the future, see these instructions.


        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

@Azizah2023 Azizah2023 force-pushed the perf/kanvas-lazy-load branch from 0e6d3de to 8d2e5ee Compare December 8, 2025 13:39
@l5io
Copy link
Contributor

l5io commented Dec 8, 2025

🚀 Preview for commit 8d2e5ee at: https://6936d8527994ab06e3a9817b--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Dec 8, 2025

🚀 Preview for commit 93b6fde at: https://6937065250b01e7f4f98ee18--layer5.netlify.app

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) issue/dco Commit sign-off instructions project/kanvas project/sistent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants