social-proof-section-master
Design comparison
Community feedback
- @Victor-NyagudiPosted almost 2 years ago
Great work on this one, Felice.
The solution looks almost exactly like the design on desktop and mobile. There are no HTML validation errors or accessibility ones. Keep it up!
However, I noticed that between 480px and 1110px, things start to break a little.
Part of this could be because you're using absolute units like
px
on the.container
class. These aren't the best for responsiveness because they don't adjust as the screen gets bigger or smaller.I'd suggest using relative units like
em
instead if you want a container that adjusts.Alternatively, instead of setting a height and width on an element, you could increase its padding, preferably using relative units also to make it bigger or wider.
If all this is too complex, maybe you can maintain the portrait design up until the screen is large enough to fit all three testimonials in landscape format.
Try it, and let me know how it goes,
Good luck with future solutions.
Marked as helpful0@feliceNicolasBarcellonaPosted almost 2 years ago@Victor-Nyagudi thank you for the feedback
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