NFT preview card component solution

Please log in to post a comment
Log in with GitHubCommunity feedback
- @AndresOreVel
Overall, it's pretty good. But you can achieve the same result by simplifying the code a bit. In the main image, you're applying with and a height of 302. It's not really necessary. Because later in the "card-image" class, you're already giving it a width of 100%. Personally, what I would have done would have been to give the main class a max-width of 20rem and work from there. Or, if you want to work on cards, apply that max-width to card and not to main. Overall, it's fine, but you're classifying almost every line of code, which leads to a lot of repetition.
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