Simple NFT preview card component, using a little bit of flexbox.

Community feedback

• 

  Kawsar Ahmed Fahad•2,660

  @faha1999Posted about 2 years ago

  Hello, Sebastian Congratulations on finishing this project. It's lovely and great on the whole! Just a little tip:

  • You might want to use semantic tags like the <main> to wrap your code, instead of div. like

  <main class="wrapper">
  </main>
  

  This would help improve accessibility.

  • add alt="equilibrium" attribute in img . This would help improve accessibility.

  • use modern CSS reset https://piccalil.li/blog/a-modern-css-reset/

  • add the below code to the body. It will center everything

  body {
      justify-content: center;
      align-items: center;
      display: flex;
      height: 100vh;
      flex-direction: column;
  }
  

  • remove margin: 5vw auto; in wrapper it's not necessary anymore.

  I hope it will work. Happy coding.

  Marked as helpful

  0