Align Learning Paths section with new Layer5 Academy experience#7794
Align Learning Paths section with new Layer5 Academy experience#779411yashiagrawal wants to merge 7 commits into
Conversation
|
Preview deployment for PR #7794 removed. This PR preview was automatically pruned because we keep only the 6 most recently updated previews on GitHub Pages to stay within deployment size limits. If needed, push a new commit to this PR to generate a fresh preview. |
68503d6 to
fdd5e3f
Compare
|
@KatalKavya96 Kindly review the changes made and help with any suggestions that apply. |
|
Thanks! |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR updates the “Learning Paths” areas on the Learn pages to align with the new Layer5 Academy experience by replacing dynamic learning path / metrics UI with a single, lower-maintenance illustration and CTA.
Changes:
- Removed MDX GraphQL queries and the legacy Learning Path cards / LearnInfo metrics display.
- Added a new
CanvasIllustrationcomponent (SVG/DOM-based) with a CTA linking to the Academy learning paths. - Added styled-components styling for the new illustration, including responsive and theme-aware styling.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| src/sections/Learn/LearnPage-Sections/learning-path.js | Replaces queried cards with the new illustration component. |
| src/sections/Learn-Layer5/index.js | Removes query + legacy cards/metrics and embeds the new illustration. |
| src/components/Learn-Components/LearnInfo/index.js | Deletes legacy metrics component (no longer used in Learn page). |
| src/components/Learn-Components/LearnInfo/learninfo.style.js | Deletes legacy metrics styling. |
| src/components/Learn-Components/Canvas-Illustration/index.js | Adds the new illustration + CTA component. |
| src/components/Learn-Components/Canvas-Illustration/canvas-illustration.style.js | Adds styles/animations and responsive behavior for the illustration. |
KatalKavya96
left a comment
There was a problem hiding this comment.
@11yashiagrawal Please address all the Copilot review comments,as they all look important for accessibility, rendering reliability, and implementation quality.
Once you have fixed them, please resolve those review comments accordingly.
Also, the newly added design section does not seem properly aligned with the other components on the page. Could you please adjust its alignment so it stays visually consistent with the rest of the Learn page layout?
Rest of the direction looks good to me once these points are addressed!
|
@11yashiagrawal Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂 |
Signed-off-by: 11yashiagrawal <11yashiagrawal@gmail.com>
…on effects Signed-off-by: 11yashiagrawal <11yashiagrawal@gmail.com>
Signed-off-by: 11yashiagrawal <11yashiagrawal@gmail.com>
…e node icons, and adjust styles for improved layout and responsiveness Signed-off-by: 11yashiagrawal <11yashiagrawal@gmail.com>
d2dfadb to
b27fe61
Compare
The illustration finally looks like this after the edit, incorporating logos and icons familiar to meshery and highlighting the new Academy experience. |
|
This looks good to me, but I think it should be reviewed by a maintainer before merging. |
|
@11yashiagrawal, how will I access the content on Layer5.io? I'm seeing redirects to cloud.layer5.io instead. Is there another way to access the content, or is this redirect expected? |
|
@saurabhraghuvanshii Yes, this redirect is intentional Based on the discussion and recommended approach in the issue, the idea is to keep the Learning Paths section on Layer5.io as a simple entry point to Academy, instead of maintaining the same content in two places. The button now takes users to the Academy page, where the learning path content is already filtered and available to explore directly. This should help keep the main site easier to maintain while still giving Academy proper visibility. |

Description
This PR resolves #6738.
Notes for Reviewers
Summary
In accordance with the discussion outlined in the issue and subsequent comments, this implementation follows Option 3. The existing Learning Paths section has been simplified by replacing the previous content with a single, low-maintenance visual element accompanied by a clear call-to-action (CTA) that directs users to the Layer5 Academy Learning Paths page.
Changes Made
Before:
After:
Signed commits