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

Ritai#4 #14

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

Ritai#4 #14

wants to merge 21 commits into from

Conversation

tjh2822
Copy link
Member

@tjh2822 tjh2822 commented Oct 22, 2017

We've got some content!

My thoughts: I like the carousel, the paragraph need some work (and images to go along with them). The background color is somewhat gaudy - let's work on that. I want to get another paragraph with out github logo and slack channel like so:

ugly

We also need to fix the issue I'm having with the navbar where there is a little gap. I'm not sure how to fix it but the issue goes away when we remover the navbar in it's entirety. It's hideous, see below:

ugly2

@tjh2822 tjh2822 added the review label Oct 22, 2017
@tjh2822 tjh2822 requested review from DBowald and adg4323 October 22, 2017 21:38
@mxnavid
Copy link
Collaborator

mxnavid commented Oct 29, 2017

Looks good!

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.

3 participants