My solution

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Greetings, Yuniac! 👋

  Well done on this challenge! 👍 Your solution looks good and is responsive! 🙌

  I'd tiny thing or two I suggest,

  • Perhaps making the font-size of the heading a little bigger and the font-size of the paragraph beneath it a little smaller so that there's some more contrast between the two (as in the original design). 😉
  • Setting text-align to center for the paragraph beneath the heading in the mobile-friendly layout of the site.

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Yuniac•65

  @YuniacPosted almost 4 years ago

  @ApplePieGiraffe Thank you, will definitely apply your recommendations and see how it turns out, thanks again appliepiegiraffe creature!

  0
• 

  Puwiii•190

  @puwiiiPosted almost 4 years ago

  Hi Yuniac!

  I can see that you have a problem with the footer ul element (Facebook, Twitter, and Instagram logos) there are not centered. You can add this in the media query: padding: 0;

  Have a great time!

  1

  ---

  

  Yuniac•65

  @YuniacPosted almost 4 years ago

  @puwiii Thank you, will try to apply it and see how it plays out.

  0
• 

  Anna Leigh•5,135

  @brasspetalsPosted almost 4 years ago

  Hi, Yuniac! 👋

  background-size: cover should definitely help you out here, but you could also play around with using background-size: contain too.

  1

  ---

  

  Yuniac•65

  @YuniacPosted almost 4 years ago

  @brasspetals Thank you, will test it out!

  0