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

about us how to join #133

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

about us how to join #133

wants to merge 3 commits into from

Conversation

cabel200
Copy link
Collaborator

No description provided.

@oupadhyay oupadhyay added the styling Something doesn't look right label Jun 16, 2020
@sanjayyepuri
Copy link
Collaborator

How do you feel about moving the "Contact Us" section to the top of the page above the "Who We Are"?

@cabel200
Copy link
Collaborator Author

Just moved the Contact Us section to the top!

@sanjayyepuri
Copy link
Collaborator

The headings seem to have gotten mixed up. It might help to remove the "Who We Are" and then make the "Contact Us" bigger to match "About Us".

Screen Shot 2020-06-28 at 12 56 50 PM

@DheekshuKumar
Copy link
Collaborator

@sanjayyepuri was this good? It looks good to me

@johancc
Copy link
Collaborator

johancc commented Jul 19, 2020

@DheekshuKumar Made some minor changes, I think this should be merged. However, the name of the nav link is "About Us", should we include the about us paragraph from the homepage?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
styling Something doesn't look right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants