Fix: Discussion cards non-functional — wire onClick to redirect to GitHub discussion

[Copilot]

DiscussionCard had cursor: pointer CSS but no click handler — only the title <a> tag was interactive. Clicking anywhere else on the card (body text, author, engagement stats) did nothing, making cards appear broken.

Changes Made

• src/components/discussions/DiscussionCard.tsx
  • Added handleCardClick → window.open(discussion.html_url, "_blank", "noopener,noreferrer")
  • Bound to card wrapper onClick so the entire card surface is clickable
  • Added e.stopPropagation() on the inner title <a> to prevent double-navigation
  • Keyboard accessibility: role="button", tabIndex={0}, onKeyDown handles both Enter and Space, aria-label

// Before: only the title <a> was interactive
<motion.div className="discussion-card-new" ...>
  <a href={discussion.html_url} target="_blank">...</a>

// After: entire card is clickable
<motion.div
  className="discussion-card-new"
  onClick={handleCardClick}
  role="button"
  tabIndex={0}
  onKeyDown={(e) => (e.key === "Enter" || e.key === " ") && handleCardClick()}
  aria-label={`Open discussion: ${discussion.title}`}
>
  <a href={discussion.html_url} target="_blank" onClick={(e) => e.stopPropagation()}>

Dependencies

• No new dependencies.

Checklist

• My code follows the style guidelines of this project.
• I have tested my changes across major browsers and devices
• My changes do not generate new console warnings or errors .
• I ran npm run build and attached screenshot(s) in this PR.
• This is already assigned Issue to me, not an unassigned issue.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
recode-website	Ready	Preview, Comment	May 4, 2026 11:45am

[sanjay-kv]

@Adez017 the issue raised and what it done is different right, there is already redirection to github working?

[Adez017]

@Adez017 the issue raised and what it done is different right, there is already redirection to github working?

Currently only the title is redirecting and now this creates whole cards functional. Isn't it ?

[sanjay-kv]

thanks for clarification

[github-actions]

🎉 Your pull request has been successfully merged! 🎉 Thank you for your valuable contribution to our project.

I would be grateful if you can nominate for keeping this opensource project live and running Link to Nominate. Please do star the repo and Keep up the fantastic work! 🚀