Design comparison
Solution retrospective
I'm a real newbie in front. If you have any comments, don't hesitate. What did I do right or wrong? What could I improve, tips and tricks !?
Community feedback
- @azhar1038Posted over 2 years ago
Hi Mamadou,
Awesome work there! I noticed that on smaller screen just before it moves to mobile view, your ratings are spreading out on the right hand side a lot thus creating a huge gap on the right side on 3rd review card. Maybe you can try centering them a bit. You may also wite a media query for smaller screen that will make them all start from same position.
Also when using images, do write
alt
too. If is just a decorative image, you can just writealt=""
or even usearia-hidden
Marked as helpful0 - @correlucasPosted over 2 years ago
๐พHello Mamadou, congratulations for your solution!
I saw your code to see what you can do better, and everything is fine, you did pretty much everything.
My only advice for your is to keep consistent the design elements, for example, the border-radius should be the same for the rating section, you've used
1rem
. You should follow the same rounded border you've used in the testimonial sections that are0.5rem
.Here's is the section I'm talking about:
.customer-card { border-radius: 0.5em; }
The rest is really good, you've a fluid and responsive solution!
I hope it helps, keep it up Mamadou!
Marked as helpful0
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