Skip to content
Submitted over 2 years ago

Responsive NFT preview card component

Jose Rodrigo•70
@nonamehz
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

I did my best 😁

Community feedback

  • VicDev ☪︎•330
    @ViicDevPosted over 2 years ago

    nice job @nonamehz 👏 just a small detail and that is to add a touch of shadow to the card

    Marked as helpful
    0
    Jose Rodrigo•70
    @nonamehzPosted over 2 years ago

    @PitagoraStrike thanks :D

    0
  • Travolgi 🍕•31,400
    @denieldenPosted over 2 years ago

    Hi Jose, I took some time to look at your solution and you did a great job!

    Also add main tag and wrap the card for Accessibility

    Overall you did well :)

    Hope this help and happy coding!

    Marked as helpful
    0
    Jose Rodrigo•70
    @nonamehzPosted over 2 years ago

    @denielden thanks! :D

    1
  • Naveen Gumaste•10,480
    @NaveenGumastePosted over 2 years ago

    Hello Jose Rodrigo ! Congo 👏 on completing this challenge

    Let's look at some of your issues, shall we:

    • you can find text font-size and style in style.guide.md so fix that

    • Always use h1 first and then h2, h3 and so on,you have used h3

    • Warp your card content around the main tag Ex: 👇

    <body>
<main class="container">
*all you content here*
</main>
</body>

    happy Coding😀

    Marked as helpful
    0
    Jose Rodrigo•70
    @nonamehzPosted over 2 years ago

    @Crazimonk thanks for your feedback :), i'm fixing it.

    1

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