Skip to content

Conversation

shristhi-2908
Copy link
Contributor

Which issue does this PR close?

Rationale for this change

Each page had its own hardcoded navbar, which caused duplication and inconsistent design.
This PR makes the navbar reusable and centralized, so future changes are easier and consistent.

What changes are included in this PR?

  • Created components/navbar.html
  • Added loadNavbar.js to dynamically load the navbar into all pages
  • Linked a CSS file for consistent styling across all pages (Home, About, Contact, etc.)

Are these changes tested?

Yes.

  • Opened index.html, about.html, and contact.html in the browser
  • Verified the navbar loads correctly on each page
  • Verified the styles are consistent and links work as expected

Are there any user-facing changes?

Yes.

  • Users will now see the same consistent navbar across all pages

No breaking changes. The PR only affects the navbar and does not break other functionality.

Screenshots

Screenshot 2025-09-24 175518 Screenshot 2025-09-24 175529 Screenshot 2025-09-24 175539

Copy link

netlify bot commented Sep 24, 2025

Deploy Preview for growcraftt ready!

Name Link
🔨 Latest commit 7c220c7
🔍 Latest deploy log https://app.netlify.com/projects/growcraftt/deploys/68d3e91a32f3f5000839f3a0
😎 Deploy Preview https://deploy-preview-654--growcraftt.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@gyanshankar1708
Copy link
Owner

@shristhi-2908 Ressolve Conflict

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.

Add hover effect to navbar links in Content Writing page
2 participants