testimonials-grid-section

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted about 2 years ago

  👾Hello MaxSurawut, congratulations on your new solution!

  Something you can do is to improve your html markup using meaningful tags and replacing the divs. In this case, for example the main block/div that takes all the content can be wrapped with <main> or section, if you think about = the cards you can replace the <div> that’s wrapping 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 inside bigger blocks wrapped with some better markup.

  Here's a complete guide for HTML semantic TAGS: https://www.w3schools.com/TAgs/default.asp

  ✌️ I hope this helps you and happy coding!

  0

  ---

  

  MaxSurawut•240

  @MaxSurawutPosted about 2 years ago

  @correlucas Thank for your advice I'll use it on next project Cheers

  0
• 

  Hawi Girmachew•250

  @HawigirmachewPosted about 2 years ago

  Hello, @MaxSurawut congratulation for completing this challenge. Your solution is excellent, and I learnt a lot from it, however I believe it would be better if you used semantic tags for it, such as main, section, and footer, as well as heading tags. I hope this will be useful to you.

  0

  ---

  

  MaxSurawut•240

  @MaxSurawutPosted about 2 years ago

  @Hawigirmachew Thank you so much

  0