Design comparison
Solution retrospective
Another easy challange. I did this one to teach some other students basic HTML and CSS.
Community feedback
- @darryncodesPosted almost 3 years ago
Hi Bram,
Awesome solution, well done.
I think at 1000px it might be cool to add
max-width: 30rem;
to.info-container
and.review-container
so that the design doesn't stretch across the whole viewport.All the best!
Marked as helpful1 - @MiculinoPosted almost 3 years ago
Hey @BramMortier
Good job on completing the challenge. Overall, it looks pretty good and the responsive design works as expected.
Just some suggestions:
-
The background images need some tweaking - refer to the original design
-
The three cards and the star rating cards aren't supposed to have the same line base line. They all need a slight offset - refer to the original design
-
Add spacing between the users' name and the "Verified Buyer" text
Marked as helpful1@BramMortierPosted almost 3 years ago@Remus432 I did make the card sit on a diagonal axis but this only goes for large Desktops. For laptops and smaller i simplified the design a bit. Thanks for the other suggestions will certainly fix those.
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