Skip to content
Submitted over 3 years ago

Testimonials grid section

Anna•545
@NitaLewska
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

Another challenge, quite an easy one. Any feedback is welcome! =) I used CSS Grid and a little bit of Flexbox. I've also done tablet variant, please check it if you have a tablet, as I don't have one.

Community feedback

  • Chamu•13,130
    @ChamuMutezvaPosted over 3 years ago

    The display looks good from mobile through to desktop. What i see missing is a heading element. A site generally should have a heading element

    1
    Anna•545
    @NitaLewskaPosted over 3 years ago

    @ChamuMutezva as the challenge is name "...section", I supposed it's not a full site, so I decided not to include header elements =)

    1
    Chamu•13,130
    @ChamuMutezvaPosted over 3 years ago

    @NitaLewska, that is very true. If you look into the error report, there is the pointer to include a heading in the sections. My other opinion was to include a header that is visually hidden but accessible to screen readers.

    1
    Anna•545
    @NitaLewskaPosted over 3 years ago

    @ChamuMutezva Sounds reasonable!

    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