nft-preview-card-component-main

Community feedback

• 

  Adriano•38,390

  @AdrianoEscarabotePosted almost 2 years ago

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

  <img src=`./images/image-equilibrium.jpg` alt=`` class=`rounded-3 img-fluid` width=`300px`>
  

  If you want to use an inline css code you need to use the style attribute, and then use the desired property! Although not recommended, but since it's a small project, it doesn't have that much problem!

  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="card-title py-2 hoverText">Equilibrium #3429</h1>

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  0