Design comparison
SolutionDesign
Solution retrospective
My first challenge from FrontendMentor. I would appreciate any feedback. Thanks and have a nice day!
Community feedback
- @ccreusatPosted almost 3 years ago
Hello!
Nice job!
Just a thought : I think you made the css complex with all the width: 87.5%, height calc etc...
- if you put everything inside your .nft-card-component width a padding: 24px; you can let everything breathe naturally
- i would suggest an img src for the image with max-width: 100%, not a background).
- remove all width: 85.2%; everywhere
- you can remove align-items: center on your .nft-card-component as well.
Marked as helpful0@mcdudPosted almost 3 years ago@ccreusat Thanks for the reply! I think i managed to correct all the elements you mentioned. I will try to simplify my css in future projects :)
1
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