Responsive 3 column preview card with CSS Grid

Solution retrospective
I'm not sure what to ask here, but let me just try: How good is the semantics? Is there a way I can do a thing better instead of something I used? What do I stop doing? What do I start doing ?
Please log in to post a comment
Log in with GitHubCommunity feedback
- @nottohave
Hello Supa, @Jos02378 kindly shared me this article and I thought that would help with replacing the padding px to em or rem. So check this out: em, rem, px and more.
Other things I think you should do:
- Definitely comment your CSS code more.
- Try to add more meaningful name to your class.
- Pay more attention at the design and try to match with it.
I can not find your html file so I can't say much about the semantics. I am not a great practitioner so I hope another pro can help answering your questions more specifically. But yea, awesome work! I hope these tips can help you out. Peace.
Join 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