Skip to content

Fix email icon on landing page to open mail client#561

Open
prashantkumarniraj wants to merge 4 commits intoAOSSIE-Org:mainfrom
prashantkumarniraj:fix-landing-mail-icon
Open

Fix email icon on landing page to open mail client#561
prashantkumarniraj wants to merge 4 commits intoAOSSIE-Org:mainfrom
prashantkumarniraj:fix-landing-mail-icon

Conversation

@prashantkumarniraj
Copy link

@prashantkumarniraj prashantkumarniraj commented Feb 5, 2026

This PR fixes an issue where the email icon on the landing page did not open
the user’s mail client when clicked.

The icon was previously wrapped in the Next.js Link component using a
mailto: URL, which caused the router to intercept the click and prevent the
browser from handling the mail protocol correctly.

This change replaces the framework-specific Link with a standard HTML
anchor tag, allowing the browser to handle the mailto: link as expected.
No visual or styling changes were introduced.

Fixes #560

Summary by CodeRabbit

  • Style
    • Updated social media icons: replaced Twitter icon with X brand icon across the footer and homepage components
    • Simplified responsive typography breakpoints in heading elements, removing select mobile and tablet breakpoint values
    • Restructured mailto and social media link elements from Next.js Link components to standard anchor tags for improved consistency

@vercel
Copy link

vercel bot commented Feb 5, 2026

@prashantkumarniraj is attempting to deploy a commit to the AOSSIE Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

This PR updates UI components to fix email link responsiveness issues and standardize icon usage. Changes include replacing Next.js Link components with standard anchor tags for mailto links across the Footer and index pages, updating the Twitter icon from faTwitter to faXTwitter, and simplifying responsive typography breakpoints in the about page.

Changes

Cohort / File(s) Summary
Contact Link Fixes
src/components/Footer.jsx, src/pages/index.jsx
Replaced Next.js Link components with plain anchor tags for mailto links to prevent router interception. Updated mail link structure to use direct href with proper aria-labels. Reordered and restructured social link elements for consistency.
Icon Updates
src/components/Footer.jsx, src/pages/index.jsx
Updated Twitter icon import and usage from faTwitter to faXTwitter across Footer and index components to reflect current branding.
Typography Simplification
src/pages/about.jsx
Removed responsive breakpoint classes (sm:text-6xl from ABOUT heading; md:text-6xl and md:text-5xl from GSOC CONTRIBUTION heading) while preserving other breakpoints for consistent styling.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A hop, skip, and click away—
No more routers blocking the mail's display!
Twitter now tweets with a fresh new X,
Typography trimmed, responsive and flex.
These small tweaks make our interfaces bright! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The PR introduces out-of-scope changes including Twitter icon replacement (faTwitter to faXTwitter) and Tailwind typography adjustments in the about page, unrelated to the stated objective of fixing the email icon. Remove unrelated changes (Twitter icon updates and about.jsx typography modifications) and create separate PRs for those improvements to maintain focus on the email icon fix.
Linked Issues check ❓ Inconclusive The PR partially addresses issue #560 by replacing the Link component with anchor tags and using href attributes, but the implementation uses simple mailto links instead of the proposed Gmail compose URL with security attributes. Verify whether the simple mailto approach meets requirements, or implement the suggested Gmail compose URL (https://mail.google.com/mail/?view=cm&fs=1&to=...) with target="_blank" and rel="noopener noreferrer" as specified in issue #560.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the primary change: replacing a Next.js Link with an anchor tag to allow the mailto protocol to open a mail client on the landing page.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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 and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/pages/index.jsx (1)

9-14: ⚠️ Potential issue | 🔴 Critical

faXTwitter is not available in @fortawesome/free-brands-svg-icons v6.2.1 — upgrade the package to v6.4.2+ or revert to faTwitter.

The project uses v6.2.1, but faXTwitter was only introduced in v6.4.2. Importing a non-existent icon will cause builds to fail.

This also applies to: 91-95

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.

Bug: Email icon on landing page does not trigger mail client

1 participant