Skip to content
Submitted over 2 years ago

Testimonial Grid - Traversy Media Tutorial

Zach James•60
@zach007james
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

  • Prince Roy•1,210
    @iprinceroyyPosted over 2 years ago

    That's excellent work @zach007james. You need to focus on some key points here:-

    1. There should be one heading tag on a web page, if there is no heading element requirement, then set its font size to 0. HTML- <h1></h1> CSS- h1{ font-size: 0; }
    2. The content should be wrapped inside the main landmark to avoid accessibility issues. Like this <body><main>your content goes here......</main></body>.
    3. There must be only one header element, here you should replace multiple header elements with div or section.
    4. There is no need for srcset attribute in the img tag.

    All the others are perfect. Hope it adds to your learning, Happy Coding :)

    Marked as helpful
    0
    Zach James•60
    @zach007jamesPosted over 2 years ago

    Thank you for your feedback @iprinceroyy ! I will remember to format my HTML better with correct <header>, <main>, and <footer> tags - only using these ones once and using <div> and <section> otherwise. Thank you for the heading tip... I had no idea you should only use one for accessibility.

    Thank you for taking the time to help. Your assistance is greatly appreciated!

    1

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