Insure landing page

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Greetings, yet again, Aaron Gonzalez! 👋

  Well done on this challenge! 👍 I love this challenge, too (it is one of my favorites, actually)! 😆

  I'd like to suggest,

  • Turning the navigation links and the social media icons in the footer of the page into actual links using the anchor tag.
  • Not switching to a mobile-friendly layout right at 1199px, as there's still quite a bit of room for the desktop layout or an intermediary/tablet layout and to prevent there from being too much empty space to the sides of the content right when the layout changes.

  Also, as a tip, you don't have to write the text for the navigation links in the footer of the page in all caps in your HTML. You can simply write it normally and then just add text-transform: uppercase to the <li>s in your CSS. 😉

  Keep coding (and happy coding, too)! 😁

  0