feat: Improved navLinks for mobile #1092
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Deploy Preview for taupe-cendol-f7e2bb ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Hey @VijaySamant4368, your PR looks good, just wait for some time it will be merged asap. |
|
@itsekta please add the Gssoc and the level label |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Title and Issue number
Title : Improved navigation links for mobile
Issue No. : #1090
Code Stack :
Close #1090
Description
I made the site's navigation bar responsive for different width of mobile devices. If the device with is small, I moved the buttons from navigation bar to the Hamburger Menu. Firstly the signup/login button moves to the menu, and then the view button. The hyperlinks work there as well. Finally if the screen width is too small(350px) the logo name is removed making space for the other more required items in the navigation
Video/Screenshots (mandatory)
Before:
https://github.com/user-attachments/assets/5fe9adaf-62e9-40aa-99f2-ad8e9b49548c
After:
https://github.com/user-attachments/assets/80e4a23c-ac2a-49b4-82a3-52381911a3ff
Type of PR
Checklist:
contributing.mdfile before contributingAdditional context:
##Are you contributing under any Open-source programme?
GSSOC'24VSOC'24