Responsive landing page using flexbox and media queries

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Great job making this far. As for your code I think you need to considered the following...

  • Clean the whitepsaces in your code
  • Instead of using section to wrap the content, use main tag
  • It's bad value to use <img src="images/facebook (1).png" alt="" class="footer-icon"></a> keep it clean for instance <img src="images/facebook(1).png" alt="facebook" class="footer-icon"></a>
  • Adjust the properties inside the mockups ruleset ...

  mockups {
    width: 50%;
    margin-top: 4rem;
    margin-left: 2rem;
  }
  

  • Add margin-left: 4rem; to the logo for alignment

  Overall you did well. Keep going and hope this helps!

  0