Design comparison
Solution retrospective
Quite a fun challenge. feedback appreciated.
Community feedback
- @HYDROCODERPosted over 3 years ago
Hey there! Good solution! There is just one issue I noticed with the testimonials. At around 670px, before your tablet breakpoint, the gap between the
testimony--customer
andtestimony--quote
becomes very high, and it can be from the fact that you used ajustify-content: space-between
for thetestimony--wrapper
class. Try centering the whole thing, or just reduce the gap (by adjusting the padding or the min-height you set for the wrapper). Otherwise, you have done a good job by using mobile-first workflow and putting some comments in your code. Although it works, try not to set fixed widths; do try using percent widths instead.Hope it helps :).
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