@ApplePieGiraffe
Posted
Hello, ToNisamBioJa! π
Good job on this challenge! π It's great to hear that you learned a lot and your solution looks nice and responds well! π
One small thing I'd like to suggest is to add background-size: cover
to the body
so that they background image fills up the entire area of the screen in the desktop layout. π
Also, it might be worth using padding rather than margin to add space around the content of the page and the sides of the screen because the body
of your page seems a little misplaced in the dev tools because of margin collapsing. It's not a huge deal, but it's useful to be aware of when layout things out in CSS. Learn more about margin collapsing in this great article.
Hope that helps a little. π
Keep coding (and happy coding, too)! π
Marked as helpful
@ToNisamBioJa
Posted
@ApplePieGiraffe Thank you for the advice and nice words!