Conversation
|
I'm still at that stakeholder meeting I mentioned last week 😓 I'll have a look at this PR soon. Btw: Could you let me take over the "Top Section" task? That would be nice. It looks so much fun. You can start with the "Bottom Section" task while you're waiting for my review. No time to loose 😉 Head back to the course to continue. |
ooloo-bot
left a comment
ooloo-bot
left a comment
There was a problem hiding this comment.
Nice job. I see one tiny problem though because the content container is too wide on small screens.
Check out my change requests. Afterward, continue with the course.
|
|
||
| .content { | ||
| min-height: calc(100vh - 100px); | ||
| width: 720px; |
There was a problem hiding this comment.
Set an additional max-width here to make the container responsive and you should be done 😄
|
The meeting is finally over but I need a coffee break first ☕☕☕ I'm back in 5 min to review the code changes. In the meantime head back to the course to update the bottom section PR. |
ooloo-bot
left a comment
ooloo-bot
left a comment
There was a problem hiding this comment.
Great work! 🎉🎉🎉 You can merge the PR now.
Once you merged this PR continue with the course.
2 participants
No description provided.