Testimonial Landing Page

Community feedback

• 

  yas•390

  @yosrajalaliPosted over 1 year ago

  Hi 👋 Congratulations on completing the challenge!

  just a couple of suggestions:

  • for your whole container it's better to use <main> tag instead of a <div>.main tag is an HTML5 tag which is supposed to represent the main content of website. it's better for SEO purposes. you can check this 👉**MDN**

   

  • also using an article tag is a better choice instead of using a <div> for your boxes. we use a div If the content within the element doesn't have any special meaning or when we need just a container, it's not wrong or bad to use a <div>, It's just better to use tags that fit the actual content of your website.

   

  • when we have a two dimensional layout using grid system will make it easier to design the web page without having to use flexbox (using flexbox in this case makes it hard to work with). check this if you want to know more aboutCss grid 👉MDN

  I hope you find it useful 🙂

  Above all your solution is great.

  0