-
Notifications
You must be signed in to change notification settings - Fork 125
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
[Bug]:Fix Navbar and Responsiveness #357
[Bug]:Fix Navbar and Responsiveness #357
Conversation
@shreyash3087 is attempting to deploy a commit to the pranavbarthwal's projects Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, you can contact the mentors for this project. |
@surajvast1 kindly verify the changes,also has any recent commits done by maintaners made some changes to the background color of this website,the light purple background color isnt there on currently deployed website,but if you fork this repo and run it locally there is light purple background. |
also as soon as i cloned this repo i got this error- which i fixed,it was just a duplicate assignment error- Notice CreateBrowser line was assigned multiple times,but its alright,was just a small fix,but im still unsure who changed the background color and is it intended or a mistake.If it was a mistake i can fix it,if its intended i'll leave it. |
We saw this issue .It will be fixed by today and you may pull the changes and modify your images up here |
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.
good work, just resolve the conflicts
@PranavBarthwal it should be good now,can you verify. thankyou |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This PR fixes the Navbar along with the responsiveness of the website,previously the website had problems with navbar and the responsiveness.The content on smaller screen was overflowing which was reducing the overall user experience for smaller devices.
Issue number
closes: #335
Video/Screenshots (mandatory)
Bug -
Bug.mp4
Fix-
Fix.mp4
Screenshots of Fixed Responsiveness-
Iphone SE-
Checklist:
contributing.md
file before contributingAdditional context:
N/A