Skip to content

fix: make getting started SVGs clickable - #8030

Closed
AdityaOP007 wants to merge 1 commit into
layer5io:masterfrom
AdityaOP007:fix/8011-clickable-getting-started-svgs
Closed

fix: make getting started SVGs clickable#8030
AdityaOP007 wants to merge 1 commit into
layer5io:masterfrom
AdityaOP007:fix/8011-clickable-getting-started-svgs

Conversation

@AdityaOP007

@AdityaOP007 AdityaOP007 commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

#8011

Summary

This PR fixes #8011 by making the relevant SVG illustrations on the Meshery Getting Started page clickable.

Changes

  • Made the integration and platform SVGs clickable with their respective destinations.
  • Added subtle hover/focus feedback to indicate interactivity.
  • Used Gatsby <Link> for internal navigation.
  • Kept the existing design, layout, and responsive behavior unchanged.
  • Preserved keyboard accessibility.

Testing

  • Verified the affected components locally.
  • git diff --check passes.
  • DCO sign-off is included in the commit.

Fixes #8011

Summary by CodeRabbit

  • New Features

    • Logo items now support links to internal pages and external resources.
    • Integration tiles on the home page are now clickable and direct users to relevant pages.
    • Platform and partner logos in the terminal section now link to associated resources.
    • Added hover transitions for linked logos and integration tiles.
  • Accessibility

    • Improved alternative text for select logos, including OAM, ImageHub, and SMP.

Signed-off-by: Aditya prakash jha <adityaprakashjha321@gmail.com>
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: cd6cc0a3-b3e5-41dd-a404-4a057b01d615

📥 Commits

Reviewing files that changed from the base of the PR and between 2b1df39 and d4a0b1c.

📒 Files selected for processing (5)
  • src/components/Logo-List/LogoList.style.js
  • src/components/Logo-List/index.js
  • src/sections/Home/Playground-home/index.js
  • src/sections/Meshery/Meshery-platforms/index.js
  • src/sections/Meshery/Meshery-terminal/index.js

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

Changes

Clickable navigation

Layer / File(s) Summary
Logo link rendering
src/components/Logo-List/*
LogoList wraps linked logos with Gatsby links or external anchors. Logo links now include hover transitions.
Playground integration links
src/sections/Home/Playground-home/index.js
Integration tiles now navigate to internal Meshery pages or external URLs. Hover styling is enabled.
Terminal-step logo links
src/sections/Meshery/Meshery-terminal/index.js
Platform and partner logos now link to related sections or sites. Several logo alt texts were updated.
Platform installation presentation
src/sections/Meshery/Meshery-platforms/index.js
Platform installation markup and code blocks were reformatted without changing commands or logic.

Priority: ⬇️ Low — Defer this navigation enhancement because it is a low-severity, user-facing update limited to clickable Getting Started integrations and platform illustrations.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Severity of issue fixed: Low

Merge Risk: ⚪ Minimal · up to d4a0b

The clickable navigation changes are ready to merge.

Sequence Diagram(s)

sequenceDiagram
  participant Visitor
  participant NavigationComponent
  participant GatsbyRouter
  participant ExternalSite
  Visitor->>NavigationComponent: Select logo or integration
  alt Internal destination
    NavigationComponent->>GatsbyRouter: Navigate with Link
  else External destination
    NavigationComponent->>ExternalSite: Open anchor in new tab
  end
Loading
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The pull request includes unrelated JSX reformatting in Meshery-platforms and Meshery-terminal, plus unrelated alt-text changes. These changes are outside the clickable SVG objective. Remove the unrelated formatting and alt-text changes, or move them into a separate pull request.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the primary change: making the Getting Started SVGs clickable.
Linked Issues check ✅ Passed The changes satisfy issue #8011. Relevant integration, platform, and partner illustrations now use internal or external links, and hover feedback indicates that the items are interactive.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 5…
  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Git: CodeRabbit could not clone the repository, so clone-backed analysis was skipped and this review may be incomplete. Verify repository clone access, such as SSH credentials, before requesting another full review. If clone access is intentionally unavailable, use path_filters to narrow the review scope.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for PR #8030 removed.

This PR preview was automatically pruned because we keep only the 3 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.

@dhruveshmishra dhruveshmishra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@AdityaOP007 Please be assigned first to raise a pr, the issue is assigned to @tejaswiverma121-byte, please close your pr.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI/UX] Make SVG illustrations clickable to navigate to their respective sections

2 participants