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 contributing link to the nav #97

Closed
wants to merge 1 commit into from

Conversation

rgaiacs
Copy link
Contributor

@rgaiacs rgaiacs commented Feb 4, 2017

Motivation

@ErinBecker query me and @mkuzak about if the contributing instructors was easy to find
and I think that it need more visibility on the website.

Screenshot

screencapture-localhost-4000-1486226389406

@mkuzak
Copy link
Contributor

mkuzak commented Feb 4, 2017

+1 I think at the top it's more visible, but we need to avoid catering the nav too

@rgaiacs
Copy link
Contributor Author

rgaiacs commented Feb 6, 2017

but we need to avoid catering the nav too

For mobile devices we have the navigation bar as a list.

Screenshot of navigation bar toggle off:

screenshot_2017-02-06_13-25-57

Screenshot of navigation bar toggle on:

screenshot_2017-02-06_13-26-12

@mkuzak
Copy link
Contributor

mkuzak commented Feb 6, 2017

yes, so I guess we don't need to worry about cluttering :)

@rgaiacs rgaiacs mentioned this pull request Feb 6, 2017
@rgaiacs rgaiacs closed this Mar 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants