Design comparison
Community feedback
- @Oluwatobi9034Posted almost 2 years ago
Hi....here are a few ideas... You should always add a basic css reset before you write your code, this is because elements have their default margins and paddings. 2...you can remove the margin auto and set the card class to display flex first before trying to center it by flexbox. 3... You should set the width of the card to either 90 or 100%, set the max width property in px , say something between 300 to 375px , it should be more responsive and easy for you that way. You can also increase the border radius property as well. And also for the image as well. 4... You should always set images to a 100% width , there's no need for the padding auto, and the margin as well , you should consider Putting the image in a div , before adding a margin to it. Or you can just set a good amount of padding in the card itself. The align items center it's not needed. For the h2, text align center is okay, no need for the justify content center.... I hope this helps
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