-
Notifications
You must be signed in to change notification settings - Fork 381
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
Shifted the search bar at the top #1439
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Utsavladia, Welcome to the project CalcDiverse! 🎊
Thanks for your contribution! Your effort makes this project better. Keep it up! 🙌
Please wait for the PR to be reviewed. Happy Coding!! ✨
✅ Deploy Preview for calcdiverse ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@Rakesh9100 This PR can be merged. |
It is not at all responsive, navbar responsiveness is lost @Utsavladia |
@Rakesh9100 What ever you wish i can change into that... |
We can go with this, but how and where all other options of navbar will be present @Utsavladia |
In case of large search bar as main component and all options of navbar in dropdown... The drop-down same as mobile screen view... In case of separate navbar for search bar component we can have a new navbar below the current one that will contain only the search bar |
Provide any supporting screenshots you have implemented on any website on how it will look after your changes @Utsavladia |
@Rakesh9100 |
Okay, we can do this but the home page design should not be disturbed @Utsavladia |
Fixes Issue🛠️
Closes #1347
Description👨💻
Shifted the search bar in the navbar.
Type of Change📄
Checklist✅
Screenshots/GIF📷