Responsive landing page using flexbox

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted about 5 years ago

  Awesome work on this challenge, it looks great and you've done a really good job with the overall layout and responsiveness 👍

  I've got a couple of small suggestions after taking a look at your code:

  • You're using the article element quite a bit, whereas it should typically be used to wrap around content that could live independently of the site it appears on, i.e. an article or blog post. Instead, I'd recommend using the section element to break your content into obvious sections by content topic.
  • You've left the alt attributes for the company logos empty. Always be sure to add alternative text in to make the images accessible to screen reader users.

  Keep up the great work!

  0

  ---

  

  adfrontenddev•70

  @adfrontenddevPosted about 5 years ago

  @mattstuddert Thank you for the feedback

  0