Skip to content
Submitted 11 months ago

NFT Preview Card Component

ra001•100
@RPRd001
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...NFT preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello

I found difficulties trying to make the component smaller once I finished it so it could be seen normally on big screen devices (like laptops or monitors). I appreciate any advice about it and any other suggestion/recommendation on my code.

Happy coding! Thanks!

Community feedback

  • Daniel 🛸•44,270
    @danielmrz-devPosted 11 months ago

    Hello @RPRd001!

    Your solution looks excellent!

    I have just one suggestion about your code:

    • You set this to the main tag:
    #main {
    height: 100vh;
    min-height: 100vh;
}
    • You don't need to use both. I'm sure that's just a detail that you missed, but use min-height instead of height. Using height may cause bugs as well as the content sometimes being cut off on smaller screens.

    I hope it helps!

    Other than that, you did a great job!

    Marked as helpful
    0
    ra001•100
    @RPRd001Posted 10 months ago

    @danielmrz-dev

    Thanks! I will correct that in the code :-)

    0

Please log in to post a comment

Log in with GitHub

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