-
Notifications
You must be signed in to change notification settings - Fork 8
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 layout #291
Add layout #291
Conversation
ea59b8f
to
4e3e06a
Compare
4e3e06a
to
735069f
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.
- On the top left below "CeSIUM" the role shows up as "role". I don't know if you forgot or weren't planning on doing it now, but I think it should be done in this PR
- I am not a fan of the mobile design. I think we can remove the organization part on top. It is taking up valuable space. When the hamburger menu is clicked, the navbar should take up the entire screen (like we do in CoderDojo Braga, and integrate the organization there
- We need to add logos for organizations
8d57cd6
to
d852cc5
Compare
21d4704
to
ee6a7ac
Compare
e662e62
to
4f69414
Compare
55807e6
to
b804ae5
Compare
Although there's an item not done, i made this ready for review , because a member of our team said he wanted to take care of that issue. |
4166a87
to
42b15dd
Compare
42b15dd
to
9531d16
Compare
8d10587
to
6c97f7b
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.
Excellent work 🚀
There's a few things missing in this PR: