Conversation
RPerry
left a comment
RPerry
left a comment
There was a problem hiding this comment.
I love the nav and title part of your code. I didn't really comment on the team class as I know you're still working on it but overall, it looks great!
index.html
Outdated
| <span>Dee Dugong</span> | ||
| <span>Startup Enthusiast</span> | ||
| </li> | ||
| <div class="first-3"> |
There was a problem hiding this comment.
I like how you separated the first 3 team members into a different class to keep them on separate lines. Was this so that the two classes stay on different lines regardless of the size of the browser?
| .container{ | ||
| display: grid; | ||
| grid-template: auto 2fr 2fr 2fr 2fr 2fr 3.5fr 2fr auto / 2fr 1fr 1fr; | ||
| grid-template-areas: |
There was a problem hiding this comment.
I'll definitely try using grid-template-areas next time
|
|
||
| } | ||
|
|
||
| .tagline { |
There was a problem hiding this comment.
Your nav and tagline classes are designed so well and look great in the browser!
| font-style: italic; | ||
| } | ||
|
|
||
| blockquote:before { |
There was a problem hiding this comment.
I had no idea you could add quotes in css
main.css
Outdated
|
|
||
| } */ | ||
|
|
||
| .why ul li{ |
There was a problem hiding this comment.
If you use vertical-align: top, you can align strong and p to be right over each other.
main.css
Outdated
| border-radius: 50%; | ||
|
|
||
| } | ||
| .first-three { |
There was a problem hiding this comment.
I think if you put the flex-direction directly within the classes, it'll put the first 3 team-members on the first row and the rest on the second row.
2 participants
Startrly
Congratulations! You're submitting your assignment.
Comprehension Questions