Skip to content

Conversation

@aniket866
Copy link

@aniket866 aniket866 commented Nov 8, 2025

As this is just a 5 lines change so I created PR directly as per rule and regulations so, please @chandansgowda merge this PR or respond if any Problem

📝 Description

This PR fixes the inactive “Contribute on GitHub” button on the Landing Page.
The button now correctly redirects users to the official AOSSIE repository:
👉 AOSSIE-Org/InPactAI

Additionally:

  • Improved button accessibility (keyboard and screen-reader friendly).
  • Ensured consistent SVG rendering and hover effects.
  • Verified the link opens safely in a new tab.

🔧 Changes Made

  • Updated the button logic in LandingPage/src/components/github.tsx.
  • Added external window.open handler to redirect to GitHub.
  • Verified the component’s functionality and styling consistency.
  • Confirmed no changes were made to other folders or files.

📷 Screenshots or Visual Changes

Before:
The button was visually present but inactive.
Uploading before contrb git button.webm…

After:
Clicking the button now opens the InPactAI GitHub Repository in a new browser tab.

(You can attach your screenshot here if you’d like — e.g., the working button UI.)

after.fix.git.button.webm

🤝 Collaboration

Solo contribution by @aniket866.


✅ Checklist

  • I have read the contributing guidelines.
  • I have manually tested the button functionality.
  • I have verified that only one file was changed (`LandingPage/src/

Summary by CodeRabbit

  • Bug Fixes
    • GitHub button on the landing page now reliably opens the repository in a new browser tab when clicked, restoring expected click behavior.
    • Fix ensures consistent experience across browsers and removes confusion where the button previously appeared non-interactive.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 8, 2025

Walkthrough

Adds an onClick handler to the GitHub button component that opens the InPactAI GitHub page in a new tab with a no-referrer policy; minor indentation change and a duplicated component declaration remain in the diff.

Changes

Cohort / File(s) Summary
GitHub Button Enhancement
LandingPage/src/components/github.tsx
Added a local openGithub function that calls window.open(url, '_blank', 'noopener,noreferrer') (no-referrer) and wired it to the GitHub button's onClick. Minor indentation changes; duplicate component declaration present in the file.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor User
    participant UI as GitHub Button (UI)
    participant Browser as Browser Window
    Note over UI,Browser `#E8F6FF`: New flow: explicit onClick handler
    User->>UI: Click GitHub button
    UI->>Browser: window.open("https://github.com/InPactAI", "_blank", "noopener,noreferrer")
    Browser-->>User: New tab opens (no referrer)
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

  • Check and remove any accidental duplicate component declaration to avoid scope/syntax issues.
  • Verify the window.open call uses the intended feature string (noopener,noreferrer) and that target/_blank semantics are correct.
  • Confirm accessibility (keyboard activation/aria attributes) and that replacing default behavior doesn't regress expected link behavior.

Poem

🐰 I hopped to a button, a tiny delight,
I clicked and it opened a new tab so bright,
No referrer trailing, the path kept quite neat,
Two copies of me—one will retreat! 🎉

Pre-merge checks and finishing touches

✅ 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 'Fix contribute on GitHub button' directly and specifically describes the main change in the pull request, which is fixing the inactive 'Contribute on GitHub' button to correctly redirect users to the repository.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 97d6c5a and d7a6612.

📒 Files selected for processing (1)
  • LandingPage/src/components/github.tsx (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • LandingPage/src/components/github.tsx

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.

@aniket866
Copy link
Author

@coderabbitai review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 8, 2025

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link
Contributor

@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: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bb74b9a and 97d6c5a.

📒 Files selected for processing (1)
  • LandingPage/src/components/github.tsx (1 hunks)

@aniket866
Copy link
Author

@coderabbitai review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 8, 2025

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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.

1 participant