Skip to content

fix: card clickable issue for Integrations#7399

Open
saurabhraghuvanshii wants to merge 9 commits intolayer5io:masterfrom
saurabhraghuvanshii:fix
Open

fix: card clickable issue for Integrations#7399
saurabhraghuvanshii wants to merge 9 commits intolayer5io:masterfrom
saurabhraghuvanshii:fix

Conversation

@saurabhraghuvanshii
Copy link
Member

@saurabhraghuvanshii saurabhraghuvanshii commented Feb 3, 2026

Description

The component cards in ComponentsGrid.js were rendered as plain <div> elements without any hyperlink or click handler. They were purely display elements with no interactive functionality.

Wrapped the component card in an <a> tag that links to Layer5 Cloud

In CatalogGrid.js, there is a prop mismatch: cardLink is used, but the correct prop is onCardClick.

This PR fixes #7397

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 3, 2026

🚀 Preview for commit f00114a at: https://6981e75e60417871c2995646--layer5.netlify.app

Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 3, 2026

🚀 Preview for commit 1688cd7 at: https://6981f61e0becb80d48dff351--layer5.netlify.app

Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 3, 2026

🚀 Preview for commit 4e394c4 at: https://6981fa2097ce6b1b7b0b387e--layer5.netlify.app

Signed-off-by: Saurabh Singh <saurabhsraghuvanshi@gmail.com>
Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
@l5io
Copy link
Contributor

l5io commented Feb 3, 2026

🚀 Preview for commit c8a9b35 at: https://698208392f1ead545ce8c4cc--layer5.netlify.app

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 added the pr/hold Do not merge this PR label Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/hold Do not merge this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Integrations: Designs failing to take action on-click

4 participants