NFT Preview Card Component

Community feedback

• 

  👾 Ekaterine Mitagvaria 👾•7,860

  @catherineisonline

  Posted over 2 years ago

  Hey there, nice solution! If you want to improve your code, make sure sure to use footer tag for your attribution div.

  You can read more about semantic html here: https://www.w3schools.com/html/html5_semantic_elements.asp

  IF THIS WAS HELPFUL PLEASE MARK IT AS HELPFUL 🤩

  00
• 

  Favour•2,160

  @Nadine-Green

  Posted over 2 years ago

  HEY SOMNATH!

  I noticed that the screenshot generated does not match the actual page, what you will need to do to fix this is to generate a new screenshot, the button to do so should be located just above the screenshot itself.

  You also have some accessibility errors, I can help you fix some of them, instead of using a div for the .atrribution, you should rather use a semantic element like footer the same goes for the .container, instead of using a div it would be better to use an element like main which is more semantic. Semantic elements are helpful because it enables screen readers to be able to describe and explain your page properly to disabled people you can learn more about semantic elements here

  IF THIS WAS HELPFUL IN ANY WAY, DON'T HESITATE TO MARK IT AS HELPFUL :)

  HAPPY CODING!

  00