Design comparison
Solution retrospective
Any feedback would be appreciated.
Community feedback
- @dewslysePosted over 3 years ago
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@tomiwaorimoloyePosted over 3 years ago@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@dewslysePosted over 3 years ago@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@tomiwaorimoloyePosted over 3 years ago@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 GitHubJoin 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