Testimonials Grid Section Using Grid And Flexbox

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  👾Hello Omer, Congratulations on completing this challenge!

  Nice solution again and nice design, you've replicated it almost perfect! Here's some tips for you:

  1.Fix the container size that is a bit off, this challenge uses the standard container size that is max-width: 1115px.

  2.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

  1
• 

  Aram•280

  @OhSorrowPosted about 2 years ago

  Hey dear Omer! Congratulations for completing this challenge! 👏

  I suggest you using blockquote element, instead of using span element (which is not semantic and is an inline element) for the quote part.

  Happy coding!

  Marked as helpful

  1