-
Notifications
You must be signed in to change notification settings - Fork 315
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
Navbar #42
Comments
Hey, @psankhe28 thanks for your interest in contributing to ProjectLearn. As a starting step, please share mockup designs of your ideas regarding the Navbar. We can discuss revisions if needed, and then the development of the same can be started. |
So,I would Like to make a navbar in the left side.Initially only the icons will be displayed and on hovering the circles,we will get to see Home,Blog as a whole word like in the below picture: The design will be as follows...also i would like to add animation to the image in that pink box and style the get started button. Please suggest any revision if you want.Thanks! |
Hey, @psankhe28 looks a bit cramped - also have you considered the mobile view? A navbar redesign like the one below would be ideal. Here's the full design idea, including mobile view - a hamburger icon to expand menu elements: |
The design you have suggested looks great! If we consider my design,for the mobile view,I think we can have a hamburger icon and a side bar can appear..like the one in below picture Also,the navbar can be done for mobile view as it is mentioned above by you.Do consider my design too and please let me know.Thanks! |
@psankhe28 mobile view looks good, however - the desktop view in your design looks a bit cramped, it's better to keep the nav elements as a horizontal bar rather than taking up vertical screen space. The GitHub star count functionality and styling should be intact, as well. |
Ok fine.I would start working on the design suggested by you |
@psankhe28 any updates till now how much work is done. |
I have almost completed it.By tomorrow,i will make a pull request. |
@Xtremilicious |
Hey @psankhe28. I don't feel this is similar to the mockup that was decided upon, as above? Also a few questions: The new Navbar looks uneven (dark on light) - not in accordance with the rest of the website's feel. I hope that you learned something and appreciate the work that you did, but unless your contributions enhance the user experience, it cannot be pushed to production. What I can suggest right now - is to follow this design solution and try to incorporate it: |
Sry for the inconvenience caused. I will redesign it. |
Hey, @psankhe28 Some elements look misaligned, but let's discuss those over on the PR thread. Looks good. Please create a PR to the |
Hi! If this issue is open, I would like to work on it! |
I would like to redesign navbar.Please assign this issue to me
The text was updated successfully, but these errors were encountered: