Skip to content

made some minor changes in the navbar. - #87

Open
Chinmay-23-cpu wants to merge 3 commits into
JS-Knights:masterfrom
Chinmay-23-cpu:mynewbranch
Open

made some minor changes in the navbar.#87
Chinmay-23-cpu wants to merge 3 commits into
JS-Knights:masterfrom
Chinmay-23-cpu:mynewbranch

Conversation

@Chinmay-23-cpu

Copy link
Copy Markdown

Overview

This PR refactors the navigation bar to improve layout, responsiveness, and maintainability without changing the overall design of the website.

Changes Made

  • Replaced the old navbar layout with a cleaner Flexbox-based structure.
  • Removed conflicting legacy navigation styles.
  • Fixed alignment issues between the logo, cart, search, and menu button.
  • Repositioned the floating hamburger menu to align with the navbar.
  • Improved spacing and icon sizing for better visual consistency.
  • Added proper top spacing so the hero section is no longer covered by the fixed navbar.
  • Fixed invalid CSS media query syntax.
  • Simplified navigation CSS by removing unnecessary float-based layouts.

Result

  • Cleaner and more modern navigation bar.
  • Better responsiveness across different screen sizes.
  • Improved code readability and maintainability.
  • No changes to the catalog, product cards, footer, or existing functionality.

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.

1 participant