Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Completed navbar responsiveness For all screen sizes in desktops and tablet. #99

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

samar12-rad
Copy link

@samar12-rad samar12-rad commented Oct 8, 2024

image

Description: This pull request addresses the responsiveness issues for all navbar versions on desktop and tablet. For issue #63

Changes Made:

The existing navbar system was cluttered, containing an excessive amount of data and div elements within a limited space. The location and delivery buttons occupied significant real estate, hindering the visibility of other essential elements in the navbar.

To improve the design, I merged the delivery and location buttons, drawing inspiration from the layout of Amazon's website to achieve a more elegant appearance. Additionally, I adjusted the CSS for the search bar to ensure it remains responsive based on the available space, enhancing the overall aesthetic.
image

I implemented a three-div structure within the navbar to facilitate automatic responsiveness. Previously, each button was housed within a single navigation div, which restricted flexibility. Furthermore, I modified the appearance of the buttons and search bar to increase their visibility.
image
Left side: Logo and Your Mode of access to GMART
Center: SearchBar
Right: Complete info of customer and orders
Benefits : Provides better understanding of UI, so increases UX by separating functions

On screens smaller than 1000px, space constraints persisted; therefore, I repositioned the logo from the left side to the top of the navbar. This adjustment alleviated the feeling of congestion and created a more intuitive layout for the buttons, resulting in a cleaner and more user-friendly experience.
image
These changes contribute to a more visually appealing and functional navbar, fostering a sense of tranquility in the overall design.

Desktop and Tablet Navbar:
Fixed responsiveness issues across all versions.

Width greater than 1550 :
image

Width 1549 to 1280:
image


Width 1280 to 1020:
image

Width 1020 to 768:
image

Ensured proper alignment and spacing of navbar elements.
Implemented media queries for better adaptability on different screen sizes.

Testing:

Verified responsiveness on multiple devices and screen sizes.
Ensured functionality of all navbar links and buttons.

Related Issues:
The existing code for the navbar was highly cluttered, making it difficult to maintain and understand. I focused on enhancing consistency throughout the codebase, resulting in a more streamlined and organized navigation structure. Because of that, it was a take to make the mobile navbar, as this alone was 450 additions and deletions of code. I have started working on the mobile navbar.

It would be great if you assign me a new issue for mobile navbar please @MinavKaria?

Current progress of mobile navbar-
image
image
image

Copy link

vercel bot commented Oct 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ratna-supermarket-mini-project ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 8, 2024 6:48am
ratna-supermarket-x3ah ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 8, 2024 6:48am

Copy link

vercel bot commented Oct 8, 2024

@samar12-rad is attempting to deploy a commit to the minavkaria's projects Team on Vercel.

A member of the Team first needs to authorize it.

@samar12-rad
Copy link
Author

Navbar will be according to the colour scheme, this is just for better visualization while developing.

@MinavKaria
Copy link
Owner

Ok, You open a new issue for mobile navbar, I'll review the code in sometime and then merge it
@samar12-rad

@samar12-rad
Copy link
Author

Sure

@samar12-rad
Copy link
Author

I created #107 for mobile version

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