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

Cristian 80

@DevCris05

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?

estoy orgulloso de poder haber realizado este ejercicio y poder completarlo dando mi mejor esfuerzo

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

al momento de reajustar el grid para pantallas mas grandes el layout se agrandaba mas y el diseño se desformaba

What specific areas of your project would you like help with?

cualquier critica que me ayude a mejorar será bien recibida :)

Community feedback

P

@srijanss

Posted

  • You have used the <figure> element to wrap the <img> tag, it would have been semantically more better if you have wrapped user name and verification text under <figcaption> element
  • Similarly, instead of using <div> which is mostly used for styling purposes, it would've been semantically correct to use either <article> or <section> element for each testimonial.
  • Nice use of grid-column and grid-row for desktop layout. You can also use grid-template-areas for it. Give it a try its more clear to understand.

Congrats, its a really nice solution. Good work.

0

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