Design comparison
Solution retrospective
Any feedback is welcome!
Community feedback
- @denieldenPosted almost 3 years ago
Hi Vicente, I took some time to look at your solution and you did a great job!
Also I have some tips for improving your code:
- remove all
margin
fromcard
class because with flex they are superfluous
Tip of graphic design: With
font-family:" Big Shoulders Display ", cursive
the browser will use the Comics Sans font when it doesn't find the first font indicated (you can seen during loading) ... for the designer it's a really awful font!- I would rather replace it with a
font-family:" Big Shoulders Display ", sans-serif
much more similar to the primary font.
Overall you did well :)
Hope this help and happy coding!
Marked as helpful0 - remove all
- @NaveenGumastePosted almost 3 years ago
hay! Vicente Solis Nice work
-> One thing border-radius can be reduced
-> Size of the heading needs to be increased
If this comment helps you then pls mark it as helpful!
Have a good day and keep coding 👍!
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