Skip to content
Submitted almost 3 years ago

nft-preview-card-component-main

Madhu U•170
@Madhu-U
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

Hi, I am new to front - end development. I'm trying to practice what I learn in my course through these challenges. I would appreciate any kind of feedback.

Community feedback

  • namlh•630
    @namlh023Posted almost 3 years ago

    Hi MashX, welcome to FrontendMentor.

    You did a pretty good job but there is something I want to point out:

    • You should have a <main> tag and a <h1> tag in your HTML file (As in accessibility issues). I recommended you change <div class='container'> to <main class='cointainer'> and <h2>Equilibrium #3429</h2> to <h1>Equilibrium #3429</h1>.
    • The overlay class doesn't need to be that complicated. Check out the solution by Juliana using pseudo-element.
    Marked as helpful
    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