Hey there! I finally completed the Responsive Web Design course
I would really appreciate some feedback on my final project:
I know it doesn’t pass one of the tests. I don’t want the welcome section to be the height of the viewport all the time, so I ignored that test.
All in all it looks good.
0 is unitless. You can just write
padding: 0. A small thing and it doesn’t matter much, but I’ve seen things like this flagged in linters.
Couldn’t this be done in one
div rather than nesting two? It does this in a few places.
Thank you for your feedback!
This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.