Insure landing page w/ CSS Grid

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hi, Jesús Alejandro Atao! 👋

  Nice to see you complete another challenge! 😀 Nice work on this one! 🙌 I like the spinning animation you added to the hover state of the social media links in the footer of the page! 🤩

  A few things I suggest are,

  • Adding a minimum height to the hero section so that the background covers all of the content on all screen sizes (currently, on smaller desktop screens, it isn't doesn't quite cover all of the content).
  • Making sure the background image of the mobile navigation shows up in the tablet layout of the site (it only shows up when the screen size gets below 37.5em, for some reason).

  Keep coding (and happy coding, too)! 😁

  3

  ---

  

  Jesús Alejandro Atao•970

  @JesusAtao96Posted almost 4 years ago

  Hi ApplePieGiraffe 👋.

  Thanks for the feedback 😀

  0