Design comparison
Community feedback
- @afaiz-spacePosted over 2 years ago
Hey @Dev-saeedAli,
- add the border radius to the card.
Marked as helpful1 - @correlucasPosted over 2 years ago
👾Hello Mohammed, congratulations for your solution!
You component is too small, take a look in the comparison slide and you see that. This container has the
width
value around 900px, so to solve that you need to add it to the .card class.Note that in this case you don't need so may div to manage the content, you need only two divs under the
<section>
one for the img and other for the text content. Removing this unnecessary elements you'll have a clean code. Don't forget to replace the section with <main> for a better semantic.Hope it helps Mohammed, keep it up!
Marked as helpful0@Dev-saeedAliPosted over 2 years ago@correlucas thanks for your feedback ♡♡♡ I am not a pro member so I dont have access to the figma files but I will change the div part thank you..
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