Design comparison
SolutionDesign
Community feedback
- @denieldenPosted almost 3 years ago
Hi Sonu, congratulations on completing the challenge.
I had a look at your solution:
- add
main
tag and wrap the card for Accessibility and don't forget theimg
element must have analt
attribute with description text, it's very important! - try to remove all
margin
fromcontainer
class and use flexbox to the body for center the card. Read here -> flex guide - also set
heigth
of body to100vh
because Flexbox aligns to the size of the parent container.
Overall you did well! Hope this help ;)
Marked as helpful1 - add
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