Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Your session has expired please log in again.
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

HTML & CSS: FlexBox

@tomiwaorimoloye

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any feedback would be appreciated.

Community feedback

dewslyse 3,025

@dewslyse

Posted

Hello @ayoelx. Congrats on your submission. It is well implemented. You may want to have a second look at the testimonial cards. They have scroll bars on them.

All the best.

0

@tomiwaorimoloye

Posted

@dewslyse yh, i delibrately put the scrollbars because I didn't want the text to overflow and I also didn't want the text to alter the height of the testimonial cards when the screen is in tablet size. So my safest bet was to just use scroll bars to avoid the whole thing. I'm not sure how else I could stop the text from altering the height of it's container without using the scroll implementation. Do you have any suggestions for that?

0
dewslyse 3,025

@dewslyse

Posted

@ayoelx, I am sure there are ways to go about it without having the scrollbars. I completed the same challenge earlier today. Maybe you should take a peek to see how I went about it.

0

@tomiwaorimoloye

Posted

@dewslyse oh, I see what you did. You added an extra card and changed the format for the tablet size.

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