Skip to content
Submitted almost 2 years ago

NFT Preview Card Component Main

danyx75•110
@danyx75
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 was having a hard time wither I should use <ul> or <p> where it says "0.041 ETH" and "3 days left". I also was having a hard time putting the image icon next to the text while using the flex-box method.

Community feedback

  • Bryan King Pecho•230
    @bkpechoPosted almost 2 years ago

    Hey there good job!

    With the "0.041 ETH" and "3 days left", I think it would be better to use <span> for them as they are short texts. Also, you may wrap them together with their icons in a <div> instead of an <article>

    I hope my feedback helps! Well done for completing this challenge. HAPPY CODING 🎉

    Marked as helpful
    1
    danyx75•110
    @danyx75Posted almost 2 years ago

    @bkpecho oh okay thank you for this info I'll do that for sure! 😁

    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