Skip to content
Submitted over 1 year ago

Testimonials Grid Section Main

YoungZV•260
@YoungZV
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

Any feedback is help me to be better and better. Thank you.

Community feedback

  • Reza Sajjadian•290
    @reza-sjdnPosted over 1 year ago

    hey @YoungZV, Good Job

    I have some recommendations for you.

    • First, you set margin-top: 1rem for .article__2 through .article__5. You didn't really need that. Instead, increase the gap between articles. This causes article__1 and article__2 to not being on the same line in desktop view!
    • Second, you have multiple h1 elements and this is not allowed in semantic html! Instead use p or h2 through h6 elements.

    Except of above, You made it successfully!

    Marked as helpful
    0
    YoungZV•260
    @YoungZVPosted over 1 year ago

    HI @reza-sjdn

    Thank you for your comment. I will make changes on my codes.

    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