Skip to content
Submitted over 2 years ago

Testimonials grid section HTML CSS

Vega•130
@vega-bit
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

Solution retrospective

please review my code

Community feedback

  • javascriptooo•310
    @javascriptoooPosted over 2 years ago

    Given the accessibility issues -- you need to include at least the <main> landmark.

    The <main> landmark tag can be inserted around your content inside the <body>. Here is some documentation

    Adding the <main> landmark will remove 2 accessibility issues! :)

    Also, with accessibility, you need headings. Add an <h1> tag instead of the <p> tag.

    <p class="testimonial"> to <h1 class="testimonial">

    and stylize the text from there. That will remove the final accessibility issue. Headings are used by screen readers for people to navigate the page. An <h1> is the main topic. So users can jump to it. If you have multiple headings, the user can skip them in screen readers.

    Check out this Headings Documentation

    Hope that helps! Great work!

    Steven

    Marked as helpful
    0
    Vega•130
    @vega-bitPosted over 2 years ago

    @javascriptooo thank you for your feedback and thank you for you for your advice your feedback is very helpful, i will look back again my code

    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