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

Testimonial grid section sample page

P
yudixa 70

@yudixa

Desktop design screenshot for the Testimonials grid section coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

In this challenge, I started to learn more about grid function, and it's not as hard as I thought 🎉

What challenges did you encounter, and how did you overcome them?

Still working on the mobile version.

Community feedback

P

@DrakeHermit

Posted

Your solution is but the cards are a bit too big compared to the design, but that's not a problem at all.

Make sure to finish the mobile version of the design too, maybe going with mobile first approach for this challenge wouldn't be a bad thing.

But other than that your code seems good, but there are few things I would change.

  • Don't put your resets on html selector
  • I would have a <main></main> tag as the container for the whole thing
  • Don't use <div> to display text, use the paragraph <p> tag
  • font-family, font-size, and background-color usually go on the body {} selector
  • Make use of CSS variables since they help out a ton, here's a link that explains CSS Variables

Marked as helpful

0

P
yudixa 70

@yudixa

Posted

@DrakeHermit Thank you for your comment. As a beginner, your input is valuable to me. Appreciate it.

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