fix: improve header alignment and navigation spacing#596
Conversation
|
Great work @SagarSingh01 Loved the newer UI, The UI you added now looks better than the earlier one! Though there are some conflicts and some checks failing kindly resolve conflicts and try if this Lint check gets successful. |
|
@Aditya948351 Done |
|
Still @SagarSingh01 There are some newer conflicts as i just merged PR #600 |
|
@Aditya948351 Okay wait |
|
@Aditya948351 Done? |
a47481c
into
devpathindcommunity-india:master
There was a problem hiding this comment.
Pull request overview
This PR updates the site navbar styling to improve visual alignment/spacing and attempts to reposition a scroll progress indicator to the bottom edge of the navbar (fixing #574).
Changes:
- Refactors
Navbar.module.cssfor a redesigned navbar layout (container positioning, nav pill spacing, mobile drawer styling). - Adds a
motion.divprogress bar element intended to scale with scroll progress. - Updates
package-lock.jsonwith lockfile metadata changes (notably removinglibcconstraints on several optional platform packages).
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 7 comments.
| File | Description |
|---|---|
src/components/layout/Navbar.tsx |
Adds a Framer Motion progress bar element to the navbar markup. |
src/components/layout/Navbar.module.css |
Major navbar CSS rewrite, including progress bar styling and responsive/mobile drawer adjustments. |
package-lock.json |
Removes libc fields from multiple Sharp-related optional platform packages. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Description
Fixes the navbar header styling and scroll progress bar positioning.
Changes
Before
After
Fixes #574
Type of change
How Has This Been Tested?
Checklist