Skip to content

Update Navbar functionality#286

Open
arnab-bol wants to merge 2 commits into
hiteshpw1:mainfrom
arnab-bol:navbar
Open

Update Navbar functionality#286
arnab-bol wants to merge 2 commits into
hiteshpw1:mainfrom
arnab-bol:navbar

Conversation

@arnab-bol
Copy link
Copy Markdown

Changes:

  • Added a navbar component
  • Modified navbar behavior and layout

Purpose:

  • Improves user navigation and structure of the website

Copilot AI review requested due to automatic review settings April 19, 2026 02:25
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the navbar markup to correct the “About us” label and adjust the displayed navigation items.

Changes:

  • Corrected the “ABout us” typo to “About us”
  • Removed the “Contact us” navbar entry
  • Left an empty line within the <ul> list

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread navbar.html
<li>ABout us</li>
<li>Contact us</li>
<li>About us</li>

Copy link

Copilot AI Apr 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is an empty whitespace-only line inside the

    after the last
  • . This adds no value and can create noisy diffs; remove the blank line or replace it with an actual nav item.

    Suggested change

Copilot uses AI. Check for mistakes.
Comment thread navbar.html
Comment on lines +4 to +5
<li>About us</li>

Copy link

Copilot AI Apr 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change removes the "Contact us" navbar entry without adding a replacement, which may reduce site navigation and seems inconsistent with the PR purpose of improving navigation. If "Contact us" is still a required destination, it should remain (possibly corrected/capitalized) rather than being deleted.

Copilot uses AI. Check for mistakes.
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.

2 participants