css html

Community feedback

• 

  vcodey•270

  @v-codeyPosted over 2 years ago

  hey @liliaazz, nice effort you have put in the solution but there are a few things to note

  • I saw your your first challenge of nft card component and @Danial Den has provided really good observations.
  • you should try to incorporate those first , like adding main and footer elements
  • dont use inline css (this is considered as not good practice)
  • check style-guide and add font and colors accordingly.

  hope you find it helpful, happy coding

  Marked as helpful

  0
• 

  Caio de Alcantara Santos•250

  @caio-alcantaraPosted over 2 years ago

  Hey! You did a good job. To make your solution look more like the design, you could add some padding to the text part. Also, you can fix these accessibility issues by wraping all of the body content into the tag <main>. Good luck on your next challenges :)

  Marked as helpful

  0