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 _ Fe Mentor Challenge

@JessicaSamtani

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


Hello all, learnt Grid from the best teacher ever. Had lots of fun and grasped the concept. Still lots to learn, excited. Open to feedback. Thanks

Community feedback

@franexmo81

Posted

Nice submission, really accurate.

I've just noticed some misbehaviour in the responsiveness:

  • The cards are not vertically centred on the screen.
  • The text overflow the cards on screens between 795 and 1250 px width
  • The violet card doesn't take the full width on screens narrower than 795 px

I think that most of the above can be fixed by just doing:

  • Remove padding: 70px for main-wrapper
  • Replace height: 100vh with min-height: 100vh for main-wrapper
  • Remove grid-template-rows: 300px 300px for testimonial
  • Remove width: 300px for the violet card under 795 px width
  • Remove margin-top: 500px and margin-bottom: 500px for main-wrapper under 795 px width

I hope the above helps you in your way to become a successful developer. Good luck!

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