Responsive grid of cards in different sizes

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted about 2 years ago

  👾Hello João, congratulations on your new solution!

  Great code and great solution! I did this challenge too and know how hard it is to set up this grid layout. I think you've done a really good job building everything! Here are some tips for you:

  Your html is working but you can improve it using meaningful tags and replace the divs, for example the main div that takes all the content can be wrapped with <main> or section, about the cards you can replace the <div> that wraps each card with <article> you can wrap the paragraph with the quote with the tag <blockquote> this way you'll wrap each block of element with the best tag in this situation. Note that <div> is only a block element without meaning, prefer to use it for small blocks of content.

  This article from Freecodecamp explains the main HTML semantic TAGS: https://www.freecodecamp.org/news/semantic-html5-elements/

  ✌️ I hope this helps you and happy coding!

  Marked as helpful

  0

  ---

  

  João Oliveira•370

  @jvmdoPosted about 2 years ago

  @correlucas I appreciate your feedback. I worked out the changes except for the testimonial div because the W3C markup validation warns there is no h2 - h6 element for the article.

  0
• 

  Travolgi 🍕•31,400

  @denieldenPosted about 2 years ago

  Hello Joao, You have done a good work! 😁

  Instead of using section inside an element use div because the <section> HTML element represents a generic standalone section of a document, which doesn't have a more specific semantic element to represent it.

  Keep learning how to code with your amazing solutions to challenges.

  Hope this help 😉 and Happy coding!

  0