NFT Preview card component main

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • To center you content to your page, add the following to your Body Element:

  body {
      min-height: 100vh;
      display: grid;
      place-content: center;
  }
  

  • The NFT Alt Tag description needs to be improved upon. You want to describe what the image is; they need to be readable. Assume you’re describing the image/icon to someone. If you have any questions or need further clarification, let me know.

  • Since "Equilibrium #3429" is a heading, it needs to wrapped in an Heading Element

  • The Icons serve no other purpose than to be decorative; They add no value. Their Alt Tag should left blank and have an aria-hidden=“true” to hides it from assistive technology.

  • Wrap the "NFT image", "Equilibrium #3429" and "Jules Wyvern" in an Anchor Tags <a>. The anchor tag will allow users to click on content and have them directed to another part of your site.

  Happy Coding! 👻🎃

  Marked as helpful

  0

  ---

  

  Luis Alfredo•120

  @luiis-fuenmayorPosted about 2 years ago

  @vcarames I understand what you are telling me, thank you very much for the suggestions they are of great help for my learning

  0