Skip to content

style: fix navbar text contrast and hover visibility in dark mode#867

Open
Nishant-Babariya-21 wants to merge 6 commits into
aryandas2911:mainfrom
Nishant-Babariya-21:style/navbar-contrast-fix
Open

style: fix navbar text contrast and hover visibility in dark mode#867
Nishant-Babariya-21 wants to merge 6 commits into
aryandas2911:mainfrom
Nishant-Babariya-21:style/navbar-contrast-fix

Conversation

@Nishant-Babariya-21
Copy link
Copy Markdown

📌 Description

Fixed a UI contrast issue in the main navigation bar. When users switched to dark mode, the links had low visibility, and hovering over inactive links made the text wash out and blend into the changing background.

🔗 Related Issue

Closes #860

🛠 Changes Made

  • Added missing dark: Tailwind CSS class modifiers to the active and idle NavLink items inside Navbar.jsx.
  • Set explicit dark-mode backgrounds and hover text properties (dark:hover:text-white) so unselected items stay clearly legible.
  • Synchronized styling changes across both desktop row rendering and mobile menu drawer maps to keep layout breakpoints uniform.

📸 Screenshots (if applicable)

image image

✅ Checklist

  • Code runs locally
  • Followed project structure
  • No console errors
  • Properly tested changes
  • Linked the issue

🚀 Notes for Reviewers

Tested on local dev server (localhost:5173) across multiple screen sizes and verified that transitions work smoothly when toggling between light and dark modes.

@Nishant-Babariya-21
Copy link
Copy Markdown
Author

@aryandas2911 plz check this pr,and if there is any issue give follow up comments

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.

style: Fix Navbar text contrast and hover visibility in dark mode

1 participant