Other: Sign in Sign up buttons in Navbar
Saekit opened this issue · comments
What would you like to share?
I would like to propose some style changes to the Sign in/Sign up buttons in the navbar.
- Change "Sign in" to "Log in" for better UI/UX experience, also to match the footer
- Change the style of the Log in button to only have an underline (on hover), so it differentiates from the Sign up button
- Align all nav items
Issues:
Funny looking button behavior:
Screen.Recording.2023-10-31.at.2.30.09.PM.mov
Solution:
![Screenshot 2023-11-02 at 1 15 44 PM](https://private-user-images.githubusercontent.com/42038008/280074429-4bb1e913-08dc-4610-b104-0210ce5aaaa5.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAyNTM4NDcsIm5iZiI6MTcyMDI1MzU0NywicGF0aCI6Ii80MjAzODAwOC8yODAwNzQ0MjktNGJiMWU5MTMtMDhkYy00NjEwLWIxMDQtMDIxMGNlNWFhYWE1LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA2VDA4MTIyN1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTlhOTIyYWUyZmRiNDNmYWIwNThiNTcxMjQ1ZjNmOGFhN2Q4N2EyNjI3ZTA4M2QyOTc3MzhlZTAyZTk3YmMyYzcmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.zeLvIZozAsOqnPs5d102CfX9gTSaKCgLNdVkdEWob4s)
🥦 Browser
Google Chrome
Checklist ✅
- I checked and didn't find similar issue
- I have read the Contributing Guidelines
- I am willing to work on this issue (blank for no)
Hello @Saekit, thank you for raising the issue.
Currently, the issue is marked as https://github.com/MilanCommunity/Milan/labels/status%3A%20todo%20%E2%8F%B3 so please wait until the maintainers/owner review it and provide you with feedback/suggestions to proceed further.
Feel free to reach out to Tamal on Twitter, or drop a mail at tamalcodes@gmail.com if you think that this issue is of critical priority.
Give us a ⭐ to show some support
Happy OpenSource 🚀
Sounds good to me however, I would also like to suggest some changes.
- The buttons can be changed, instead of having two, we can have one.
- We just keep a main button, which says
Sign In
or something like that (check the screenshot below) - Also the line height probably is probably causing some issues with the misalignment.
@tamalCodes Just to clarify, are you suggesting completely removing the Sign up button from the navbar and only having a log in button that keeps the current style?
sir can you please assign this issue to me ?
@tamalCodes Just to clarify, are you suggesting completely removing the Sign up button from the navbar and only having a log in button that keeps the current style?
Yes
can i work on it as it available i know it
Hi there @tamalCodes i went through the conversation above. I guess i can the fix this issue.
Hey @tamalCodes, I've made the changes locally. Could you please assign the issue to me? It will be helpful for me to create a pull request.
Hey I would love to work on this issue.