Design comparison
SolutionDesign
Community feedback
- @afaiz-spacePosted over 2 years ago
Hey @adniya,
- add border-radius in the child class.
- remove br tags. use flex property in the article class.
0 - @correlucasPosted over 2 years ago
Hello Adniya! Congratulatios for you solution!
I think you did a great job wrapping up all content inside the container. There's only few things you can do to improve your solution.
- Setting the proper border-radius to the card, thats
border-radius: 8px;
. - Reducing the padding between the cards when the container become vertical on mobile, the actual padding between the last paragraph its too big.
You can also check the solution report panel and fix the accessibility issues making your code concise and semantic.
Keep it up!
0 - Setting the proper border-radius to the card, thats
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