Skip to content

feat: Animate Meshery architecture diagram and add section#7715

Open
shteypandey28-hue wants to merge 5 commits into
layer5io:masterfrom
shteypandey28-hue:feature/meshery-architecture-animation
Open

feat: Animate Meshery architecture diagram and add section#7715
shteypandey28-hue wants to merge 5 commits into
layer5io:masterfrom
shteypandey28-hue:feature/meshery-architecture-animation

Conversation

@shteypandey28-hue
Copy link
Copy Markdown
Contributor

Description

This PR improves the pull request template by enhancing clarity, formatting, and contributor guidance. The updated template helps contributors provide more structured information, making reviews faster and more efficient.

Changes Included

  • Refined PR description structure
  • Improved readability and formatting
  • Added clearer instructions for reviewers and contributors
  • Maintained DCO/sign-off reminder for signed commits

Notes for Reviewers

  • Verify formatting and wording consistency
  • Ensure all contribution guidelines remain accurate

Related Issue

Fixes #

Signed commits

  • Yes, I signed my commits.

Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>
@shteypandey28-hue shteypandey28-hue force-pushed the feature/meshery-architecture-animation branch from b297cbe to 4314b9e Compare May 8, 2026 18:59
…-architecture animation

Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>
Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

🚀 Preview deployment: https://layer5.io/pr-preview/pr-7715/

Note: Preview may take a moment (GitHub Pages deployment in progress). Please wait and refresh. Track deployment here

@shteypandey28-hue
Copy link
Copy Markdown
Contributor Author

Kindly review it . @saurabhraghuvanshii

Copy link
Copy Markdown
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

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

@shteypandey28-hue Add a recorded video of the output please.

@shteypandey28-hue
Copy link
Copy Markdown
Contributor Author

Meshery.animation.mp4

Hi @Rajesh-Nagarajan-11

Thanks for the review! I've attached a recorded video of the animated architecture diagram in action below.

I've also optimized the implementation—the CSS keyframe animations are now embedded directly within the native SVG <style> block and loaded via a lightweight tag. This approach provides super smooth animations and hover interactions while entirely avoiding the SVGR out-of-memory build issues we were seeing during CI deployments.

Everything is building successfully and rendering great! Let me know if you need anything else, otherwise this should be ready to merge.

Comment thread runs.json Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@shteypandey28-hue What is use of this??

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Same as below — this was an accidental commit of GitHub Actions workflow run data. It's been removed now. Sorry for the noise!

Comment thread comments.json Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@shteypandey28-hue what is purpose of this file? I'm not seeing any import of this.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Apologies, this was accidentally committed — it's a GitHub API response that got staged by mistake. I've removed it in the latest push. Thanks for catching it!

Signed-off-by: shteypandey28-hue <shteypandey28@gmail.com>
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.

3 participants