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

@AlfredoCU

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


How to improve grid css?

Community feedback

P
Grace 27,350

@grace-snow

Posted

Hi there,

I'm just previewing on mobile, but the css grid stuff looks fine to me. If you want to improve it, I'd look into area and line names next, as all the numbers (especially written in shorthand) are difficult to read. If you used grid line or area names, other developers would be able to see where everything is placed on your grid at an easy glance.

I also think your layouts would be easier if you went mobile first with css / media queries.

Only other point for improvement is on alt text. As an assistive technology user, it wouldn't tell me anything to hear 'user 1 read out as an image description. Better to use the person's name or leave alt intentionally blank.

That's all from me. Well done on. This and keep going! 👏 ☺

2

@AlfredoCU

Posted

Hey @grace-snow, Thanks for the comments--super helpful! :D

1
P
ApplePieGiraffe 30,545

@ApplePieGiraffe

Posted

Hey, Alfredo Carreón Urbano! 👋

Good work on this challenge! Your solution looks good and is responsive! 👏

I suggest,

  • Setting a max-width on the grid in the desktop layout of the site so that the content of the page doesn't look too stretched on extra-large screens.
  • Decreasing the space to the left and right sides of the grid before the layout changes at 64em would give the content of the page some more room and prevent it from looking too squeezed right before the layout changes.

Keep coding (and happy coding, too)! 😁

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