Skip to content

Improve the navbar of the website #19

Open
Dipanita45 wants to merge 2 commits into
JhaSourav07:mainfrom
Dipanita45:main
Open

Improve the navbar of the website #19
Dipanita45 wants to merge 2 commits into
JhaSourav07:mainfrom
Dipanita45:main

Conversation

@Dipanita45
Copy link
Copy Markdown

@Dipanita45 Dipanita45 commented Apr 17, 2026

Add a sticky, semi-transparent background with slight blur for a modern feel.
Increase spacing between menu items (Edit, Convert, etc.) for better readability.
Replace dropdown arrows with subtle icons for a cleaner look.
Add a hover animation (smooth fade/slide) for better interactivity
Issue No:#13

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 17, 2026

@Dipanita45 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

@JhaSourav07
Copy link
Copy Markdown
Owner

@Dipanita45
I cannot merge this PR as it is not following rules said in CONTRIBUTING.md.
Kindly write description of the issue and also in PR.

@Dipanita45
Copy link
Copy Markdown
Author

Done

@JhaSourav07
Copy link
Copy Markdown
Owner

@Dipanita45
Kindly update the issue description also.

@Dipanita45
Copy link
Copy Markdown
Author

Done

Copy link
Copy Markdown
Contributor

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

This PR aims to modernize the site navbar (Issue #13) by applying a more “glass” visual treatment, increasing spacing, and improving dropdown interactivity/animations.

Changes:

  • Updated navbar + wallet menu styling (more blur/transparency, spacing, hover/transition tweaks).
  • Restyled all desktop dropdown triggers/menus with new animation parameters and UI accents.
  • Added a new CSS block in App.css defining .navbar/.menu glassmorphism styles.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
src/components/layout/Navbar.jsx Updates the navbar, dropdown, and wallet menu UI/animations; adjusts desktop spacing and button styling.
src/App.css Adds navbar/menu CSS rules intended for a glassy navbar appearance.
Comments suppressed due to low confidence (1)

src/components/layout/Navbar.jsx:266

  • isOpen is toggled by the mobile hamburger button, but the mobile menu panel is no longer rendered (the previous <AnimatePresence> / <motion.div> block was removed). On small screens this leaves users with no navigation beyond the hamburger icon. Please reintroduce a mobile nav drawer/dropdown (using navLinks) or remove the hamburger toggle entirely.
export function Navbar() {
  const [isOpen, setIsOpen] = useState(false);
  const { address, isConnected } = useAccount();
  const { isPremium } = useSubscription();


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

Comment thread src/App.css
Comment thread src/components/layout/Navbar.jsx
Comment thread src/components/layout/Navbar.jsx
Comment thread src/components/layout/Navbar.jsx
@JhaSourav07
Copy link
Copy Markdown
Owner

@Dipanita45

Address the Copilot comments.
And change your commit message to follow the format which is described in CONTRIBUTING.md

@JhaSourav07 JhaSourav07 added the question Further information is requested label Apr 18, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 26, 2026

👷 Deploy request for quickstpdf pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 4c88dda

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants