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

Responsive testimonials grid using CSS Grid and Flexbox

P
Alice 170

@AliceeLe

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?

I'm proud to use more responsive layout element like rem. Also, custom designs for each element is pretty laborious.

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

I was having trouble over having the grid keep its ration when adjusting the padding and margin. When I switched to rem instead of px, it works. I mostly google or search w3school for similar tutorials.

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

I'm still having trouble placing the quotation mark under the testimonial bolded text. I've tried using z-index and position:relative, but my code displays it as separate divs. Furthermore, I don't know how to make the pfp images more responsive when resizing. Would appreciate much help if anyone can help.

Community feedback

N1Dovud 170

@N1Dovud

Posted

Hi! I personally handled the quotation mark by setting its position to absolute and specifying top and right properties. I also set its parent to relative so that its position is relative to its parent rather than the whole document. Apply z index of 1 to the svg and z index of 2 to the text that overlaps with the svg. I think there is much of a need for making pfp images responsive since they are almost always small and their size does not have to change really, but if you really wanted, you could give them a different width depending on the vw. I personally think the biggest problem you have is your paragraph contents whose font size depends on the vw. You probably wanted to make them responsive but as you go anywhere below 900 px, your text gets unimaginably small and thus illegible. In mobile screen, that's even worse. So, to fix it, just give a proper rem or font size in pixels. It will solve the problem. Good job, btw!

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