Sockets - Sarah#41
Conversation
dev-elle-up
left a comment
dev-elle-up
left a comment
There was a problem hiding this comment.
Awesome job! I love the colors and the style overall. Also your code is dry and easy to read. Great work.
| } | ||
| .tagline { | ||
| background: #57BC90; | ||
| display: grid; |
There was a problem hiding this comment.
Nice use of grid on the tagline section!
| @@ -0,0 +1,149 @@ | |||
|
|
|||
There was a problem hiding this comment.
Did you consider using grid for the overall structure?
| } | ||
|
|
||
| .flexbox li { | ||
| flex: 1; |
There was a problem hiding this comment.
This is something I hadn't seen before. Thanks for the learning moment. :-)
| } | ||
|
|
||
| .sponsers .flexbox, .team-members .flexbox { | ||
| justify-content: space-around; |
There was a problem hiding this comment.
The sponsors are almost touching on my browser. Was this a design choice or a browser rendering? It could be nice to give these elements a little margin. (or not)
| flex-direction: column; | ||
| } | ||
|
|
||
| .team-members .flexbox li img { |
There was a problem hiding this comment.
I think you did a great job of naming everything! It made it very easy to read your code.
2 participants
Startrly
Congratulations! You're submitting your assignment.
Comprehension Questions