Skip to content

Conversation

@prashantkumarniraj
Copy link

@prashantkumarniraj prashantkumarniraj commented Feb 3, 2026

This PR fixes the footer mail icon by linking it to a valid mailto address,
ensuring consistent behavior with other social icons.

Fixes #555

Summary by CodeRabbit

  • Style

    • Replaced Twitter icon with X across the site.
    • Simplified responsive heading sizing on the About page.
  • New Features

    • Added GitLab and Discord social links to the footer.

@vercel
Copy link

vercel bot commented Feb 3, 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 3, 2026

📝 Walkthrough

Walkthrough

This update addresses UI/UX improvements across the website by updating social media branding (Twitter to X), fixing the non-functional footer mail link, and refining responsive typography in the About page for better visual consistency and user experience.

Changes

Cohort / File(s) Summary
Social Media Icon Updates
src/components/Footer.jsx, src/pages/index.jsx
Replaced FontAwesome Twitter icon (faTwitter) with X Twitter icon (faXTwitter) to reflect current branding across footer and homepage.
Footer Mail Link Fix
src/components/Footer.jsx
Converted mail link from Next.js Link component to a standard anchor tag; reorganized and added social links (GitLab, GitHub, Discord) for consistent link handling.
Responsive Typography Adjustments
src/pages/about.jsx
Simplified responsive breakpoints by removing sm:text\-6xl and md breakpoint overrides from About and GSOC section headings, reducing unnecessary size scaling at intermediate screen sizes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰✨ A hop and a skip, the icons now align,
X marks the spot where Twitter once did shine,
Mail links now work, a functional delight,
Typography flows smooth, left and right,
Our burrow of code grows cleaner, more bright! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 3
❌ Failed checks (3 warnings)
Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The PR implements the mail icon fix from issue #555 but also includes unrelated Twitter icon updates and typography changes not required by the linked issue. The PR should focus solely on the mail icon fix mentioned in the title. The Twitter icon and typography changes are out of scope for issue #555.
Out of Scope Changes check ⚠️ Warning The PR includes changes beyond the mail icon fix: Twitter icon replacements in Footer and index components, and typography adjustments in the About page. Remove unrelated changes (Twitter icon updates and About page typography fixes) to keep the PR focused on fixing the non-functional mail icon.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title specifically mentions fixing the non-functional mail icon in the footer, which is the primary objective addressed in this changeset.

✏️ 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

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.


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.

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 improvements needed in website (broken links, spacing, typography)

1 participant