Skip to content
Submitted about 2 years ago

Testimonial Grid Section

Tenzin•140
@lekdup
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

Hello 👋,

I've completed this grid challenge, i'm looking forward for any feedbacks and advices! Thank you!

Community feedback

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

    👾Hello Tenzin, congratulations for your new solution!

    Your solution seems really good, there are two things I think you can consider changing.

    The cards are too much rounded, a better value is border-radius: 14px.

    I think you've placed wrong the tag <main> often this tag is used to show the html which is the main section not an small block what is the main thing in the page. You can replace the paragraph with the important quote text with <blockquote> insteat of using p.

    Here's a complete guide for HTML semantic TAGS: https://www.w3schools.com/TAgs/default.asp

    👋 I hope this helps you and happy coding!

    Marked as helpful
    0
    Tenzin•140
    @lekdupPosted about 2 years ago

    @correlucas Hello Lucas, thank you very much for the feedbacks! i've made those changes and going to check out the link you've sent me.

    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