Skip to content
Submitted about 2 years ago

Testimonials Page (Responsive) With Css Grid and Flex

Htoo Myat Kyaing•100
@Htoomyat-Kyaing
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Testimonials grid section
  • HTML
  • CSS
2junior

Design comparison

SolutionDesign

Community feedback

  • Lucas 👾•104,340
    @correlucasPosted about 2 years ago

    👾Hello Htoo Myat, congratulations on your new solution!

    I’ve some suggestions for you:

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

    2.Add the website favicon inserting the svg image inside the <head>. <link rel="icon" type="image/x-icon" href="./images/favicon-32x32.png">

    ✌️ I hope this helps you and happy coding!

    Marked as helpful
    0
    Htoo Myat Kyaing•100
    @Htoomyat-KyaingPosted about 2 years ago

    @correlucas Thank you for your feedback! I will try to make it better as you suggested.

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord