-
Notifications
You must be signed in to change notification settings - Fork 3
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
Header and minor ui changes #654
Conversation
8a352d3
to
1e17ddf
Compare
4507804
to
a69230f
Compare
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.
thanks for tackling this! headers are super complicated
I put this in comments but wanted to mention it here; images should live in the frontend2/public
directory. When you want to reference it in code, you just need to use /<image_name_with_extension>
to display it
Also please make sure to run npm run format
so that your code is formatted :)
a69230f
to
79631f8
Compare
consistency responsive header update header remove chatgpt comments
79631f8
to
ab1a26e
Compare
631db21
to
50ba18a
Compare
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.
LGTM! 🎉
Co-authored-by: Serena Li <serena.li.usa@gmail.com>
Co-authored-by: Serena Li <serena.li.usa@gmail.com>
Co-authored-by: Serena Li <serena.li.usa@gmail.com>
Adds the Header component (nav bar at the top of the page)
Note: we don't automatically generate profile pictures for new accounts, so the alt text shows in the image below. We should either generate this in the backend (#327 has some work on this), or display a default profile picture. I've added this onto our roadmap
Testing:
Log in
option at the bottom of the dropdown.Your profile
, andSign out
Log in
andRegister
button in the right side of the header. TheLog in
button will disappear at small window widths.