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

Add more option on navbar #162

Closed
wants to merge 5 commits into from

Conversation

pandeyji711
Copy link

@HimanshuNarware Add more option related to the website
Screenshot 2024-05-15 004921

Copy link

vercel bot commented May 15, 2024

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

Name Status Preview Comments Updated (UTC)
career-zunction ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 19, 2024 7:35am

@pandeyji711 pandeyji711 changed the title Add navbar Add more option on navbar May 15, 2024
@pandeyji711
Copy link
Author

@HimanshuNarware please review and merge

@pandeyji711
Copy link
Author

@HimanshuNarware please label it Gssoc 2024

@PSS2134
Copy link
Contributor

PSS2134 commented May 15, 2024

Which issue are you fixing @pandeyji711 ?

@pandeyji711
Copy link
Author

@PSS2134 #40 please add label Gssoc 2024

@PSS2134
Copy link
Contributor

PSS2134 commented May 16, 2024

@pandeyji711 This issue #40 is about adding icons , which cannot be seen ?

@pandeyji711
Copy link
Author

@PSS2134 Now added fabicon and more options

Recording.2024-05-16.143256.mp4

@Shubh942
Copy link
Contributor

Ensure there is adequate space between the icon and the text.

@pandeyji711
Copy link
Author

@Shubh942 fixed spacing
Screenshot 2024-05-17 132434
please review and merge with Gssoc24 and level tag

@PSS2134
Copy link
Contributor

PSS2134 commented May 19, 2024

add some space between icon and text and also align them at same level horizotally @pandeyji711

@pandeyji711
Copy link
Author

add some space between icon and text and also align them at same level horizotally

@PSS2134 fixed

@pandeyji711
Copy link
Author

@HimanshuNarware please review and merge

@HimanshuNarware
Copy link
Owner

@pandeyji711 this is not responsive and effecting the responsiveness of the Web that why closing this PR

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.

4 participants