-
Notifications
You must be signed in to change notification settings - Fork 10
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
use the new website design from ElementalCode/elemental#220 #233
Conversation
see #220 |
|
||
/* == Logged out user things == */ | ||
// Is it possible to only serve these when the user is logged out? | ||
#logged-out-header { |
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.
Could we use classes instead of IDs?
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.
Done
Other than the comment above, it looks good |
There's still a lot of stuff in style.scss that might not be used--I haven't checked |
Is there anything else you want added before merging this? |
I just forgot to merge it, oops |
I haven't done any of the logged-in stuff yet 'cause I'm having trouble registering a new account on localhost. But I broke down the HTML into navbar/index/footer/etc. and tried to do the same for the SASS, and I updated the login and signup pages.