Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Testimonials Grid Section Main

YoungZV• 260

@YoungZV

Preview site(opens in new window)View code(opens in new window)
Desktop design screenshot for the Testimonials grid section coding challenge

This is a solution for...

Testimonials grid section
  • HTML
  • CSS
2junior
View challenge

Design comparison

SolutionDesign

Solution retrospective

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

Community feedback

Reza Sajjadian• 290

@reza-sjdn

Posted

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

@YoungZV

Posted

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
Discord logo

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