NFT-preview-card-component

Community feedback

• 

  Adriano•38,390

  @AdrianoEscarabotePosted almost 2 years ago

  Hi MrSkran, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:

  To improve the design of the page we can do this:

  .avatar {
      border-radius: 10px;
  }
  

  To improve the structure of the html document, prefer to wrap all the content with the main tag, since this challenge is just a component, it needs a main tag to know what the main content of the page is!

  As you know this page is just a component, so it doesn't even need an h1! because we don't know how important the other components of the site will be! but it's always good to prevent accessibility errors so I think it would be good for you to add an h1 in this component, besides being a good practice for when you are developing larger sites, don't worry forget about h1.

  <h1 class="title">Equilibrium #3429</h1>

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  0