Skip to content

fix: unify navbar styling across Home, About and Products pages#841

Open
PARNITA-SINGH wants to merge 3 commits into
adithyan-css:mainfrom
PARNITA-SINGH:main
Open

fix: unify navbar styling across Home, About and Products pages#841
PARNITA-SINGH wants to merge 3 commits into
adithyan-css:mainfrom
PARNITA-SINGH:main

Conversation

@PARNITA-SINGH

Copy link
Copy Markdown

🔗 Fixes
Closes #801

🐛 Problem
The navigation bar rendered inconsistently across pages:

  • About and Products pages had different background colors
  • Logo styling was inconsistent across pages
  • No visual indicator for the currently active page
  • Extra/duplicate nav links on products.html
  • Conflicting inline CSS overriding global style.css

✅ Changes Made

public/about.html

  • Removed conflicting inline header CSS overrides
  • Fixed logo to use .logo-name and .logo-sub classes matching index.html
  • Added class="active" to About nav link
  • Fixed misplaced HTML inside <style> block

**public/products.html**

  • Removed inline header background override
  • Removed conflicting header nav a.active inline styles
  • Fixed class="acstive" typo → class="active" on Products link
  • Cleaned up duplicate/extra nav links

public/style.css

  • Added nav a.active styles for consistent active page indicator
  • Standardised nav a::after underline color to gold

📸 Before & After
image

image image

@vercel

vercel Bot commented Jun 14, 2026

Copy link
Copy Markdown

@PARNITA-SINGH is attempting to deploy a commit to the adithyansubramani1-1657's projects Team on Vercel.

A member of the Team first needs to authorize it.

@PARNITA-SINGH

PARNITA-SINGH commented Jun 14, 2026

Copy link
Copy Markdown
Author

Hey! I've been working on this issue and have made the following fixes I raised from Issue , please Authorize on Vercel so I can complete all the checks that are required to deploy.

@PARNITA-SINGH

Copy link
Copy Markdown
Author

Hi @adithyan-css

I submitted this PR and all checks have passed except the Vercel deployment check, which is currently waiting for authorization.
Could you please authorize the Vercel deployment and review the PR when convenient?

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] - Navbar appearance changes between Home, About, and Products pages

1 participant