Skip to content

[GOOD FIRST ISSUE] Add hover states to Navbar links#481

Merged
PRODHOSH merged 1 commit into
PRODHOSH:mainfrom
hanu-14:feat-navbar-hover-281
Jul 18, 2026
Merged

[GOOD FIRST ISSUE] Add hover states to Navbar links#481
PRODHOSH merged 1 commit into
PRODHOSH:mainfrom
hanu-14:feat-navbar-hover-281

Conversation

@hanu-14

@hanu-14 hanu-14 commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Closes #281

Added hover:text-primary with transition-colors to Navbar links.

Labels: ELUSOC NEWBIE

Summary by CodeRabbit

  • Style
    • Improved hover effects and transition animations for the desktop navigation, theme toggle icons, and mobile menu button.

@github-actions

Copy link
Copy Markdown
Contributor

Welcome to OSSfolio, @hanu-14! 🎉

Thank you for opening this pull request and contributing to the open-source community! 🚀

To ensure a smooth review process, please make sure you have:

  • Checked that your changes work locally and compile cleanly.
  • Followed the guidelines outlined in CONTRIBUTING.md.
  • Linked your PR to an open issue (e.g. Closes #ISSUE_NUMBER).

We will review your PR as soon as possible. Happy coding! 💻✨

@github-actions github-actions Bot added frontend Related to UI / Next.js UI Visual / design changes labels Jul 18, 2026
@coderabbitai

coderabbitai Bot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 251d2854-a346-40df-8e87-df61a66c966b

📥 Commits

Reviewing files that changed from the base of the PR and between 5d9f7cb and 7247c0d.

📒 Files selected for processing (1)
  • src/components/layout/Navbar.tsx

📝 Walkthrough

Walkthrough

Navbar class names now add hover colors and transition styling to desktop navigation, theme controls, and the mobile menu toggle.

Changes

Navbar hover styling

Layer / File(s) Summary
Hover and transition classes
src/components/layout/Navbar.tsx
Desktop navigation, theme toggle icons, and the mobile menu toggle gain hover text color and transition timing classes.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested labels: ELUSOC, frontend, UI, completed, NEWBIE

Suggested reviewers: sakethsumanbathini, asiya2123, prodhosh

Poem

I’m a bunny with a bright UI,
Hopping where hover colors fly.
The moon and sun now softly gleam,
Menus transition like a dream.
Carrots up for polished flair! 🐇

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (3 warnings)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The changes add hover styling to Navbar controls, but the requested hover states for the navigation Link elements aren't clearly shown. Update the actual Navbar Link tags with a Tailwind hover class like hover:text-primary or an underline, and verify the link hover behavior.
Out of Scope Changes check ⚠️ Warning The PR also styles the theme toggle icons and mobile menu button, which are outside the link-hover scope in #281. Keep the PR focused on Navbar links, or move the extra icon/button hover styling into a separate change.
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 title is concise and clearly describes the Navbar hover-state styling change.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@PRODHOSH
PRODHOSH self-requested a review July 18, 2026 04:29
@PRODHOSH PRODHOSH added ELUSOC ELUSOC project submission completed NEWBIE Beginner — 10 pts beginner Good for newcomers labels Jul 18, 2026
@PRODHOSH
PRODHOSH merged commit 6317fd9 into PRODHOSH:main Jul 18, 2026
8 of 10 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

🎉 Your PR just got merged, @hanu-14 — thank you for contributing to OSSfolio!

Your work is now part of the project. Here's what to do next:

  • ⭐ If you haven't already, consider giving the repo a star — it helps us grow.
  • 📢 Share your contribution on LinkedIn, Twitter, or wherever you hang out. You shipped open source!
  • 🔍 Browse other open issues if you want to keep contributing.

We really appreciate you taking the time. See you in the next PR! 🚀

@PRODHOSH

Copy link
Copy Markdown
Owner

@all-contributors add @hanu-14 for code

Im adding u to the contributors list in readme
incase of doubts or queries u can tag me here or in official discord from #34
star the repo

Also Please read the CONTRIBUTING.md and #19
ur supposed to use the pr template given

and ur supposed to get assigned to an issue before u raise a pr
these are common open source guidelines to follow

@allcontributors

Copy link
Copy Markdown
Contributor

@PRODHOSH

I've put up a pull request to add @hanu-14! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beginner Good for newcomers completed ELUSOC ELUSOC project submission frontend Related to UI / Next.js NEWBIE Beginner — 10 pts UI Visual / design changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[GOOD FIRST ISSUE] Add hover states to Navbar links

2 participants