Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

NFT Card

@Abdul-ganuine

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


This is my second project any advice or suggestion is very welcome

Community feedback

@dubervelez

Posted

Hi Abdul Ganuine. Good job, according to the responsive design I have some suggestions:

-You are using two containers that have the class "card" and "container", the "container" class is only used to have a margin between the content and the container where you use the margin and width properties of the class. You could eliminate that class and use only with the "card" class and to make that separation between the content you use the property padding: 25px.

-To improve the responsive I recommend you modify the height property to "auto" for the "card" class container, because there are resolutions where the content goes out of the container, the other property to modify would be to add a min-width, it can be 260px to make it look better on different devices.

good day!

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

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