Skip to content
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 user menu #22

Merged
merged 2 commits into from
Dec 8, 2017
Merged

add user menu #22

merged 2 commits into from
Dec 8, 2017

Conversation

kthull
Copy link
Contributor

@kthull kthull commented Dec 3, 2017

Adds the top-header user menu per the 2017 site style. Some considerations:

  • this does not include the camp date/location because it lives in the even banner; depending on how the header is treated on internal pages, we may need to add this back
  • last year's main nav had a log in link at mobile, where this year's does not, so the user menu remains at smaller breakpoints

@kthull
Copy link
Contributor Author

kthull commented Dec 3, 2017

We can also add Log in and Log out links to the main nav as the first link and hide first child at non-mobile breakpoints. Then on not-front, we can minimize or ditch the event banner and then hide the user menu in the top-header and instead show a date/location.

@zendoodles zendoodles merged commit e4fb200 into develop Dec 8, 2017
@zendoodles zendoodles deleted the user-menu branch December 8, 2017 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants