Frontend Mentor - NFT preview card component

Community feedback

• 

  Gaurav Pratap Singh•340

  @gauravsingh1281Posted about 2 years ago

  Hii Chiemeka Okongwu, Congratulations on completing this challlenge 👏👏. I have a suggestions for you. You didn't use the hovering effect and cursor pointer property on Equilibrium image, Equilibrium text and on the avatar's name as mentioned in this challenge when a user hover this element there should be a hovering effect and cursor which make your website more interactive.

  Marked as helpful

  0

  ---

  

  Chiemeka Okongwu•550

  @Chiemeka2006Posted about 2 years ago

  @gauravsingh1281 thanks for the feedback ill fix it up later i didnt read the read me stuff but illl get it done

  0
• 

  Adriano•38,160

  @AdrianoEscarabotePosted about 2 years ago

  Hi Chiemeka Okongwu, 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 accessibility of the project you could have put an h1. Every page must contain a level 1 header, for people who use screen readers, identity what the main title is.
  • every Html document must contain the main tag, so we can identify the main content, to fix this, wrap all the content with the main tag.

  The rest is great!

  I hope it helps... 👍

  0

  ---

  

  Chiemeka Okongwu•550

  @Chiemeka2006Posted about 2 years ago

  @AdrianoEscarabote it helps a lot thanks for the feedback i really appreciate it im new to css and html so im not too used to the html semantics but thanks for correcting me

  1

  

  Adriano•38,160

  @AdrianoEscarabotePosted about 2 years ago

  @Chiemeka2006 happy coding!!

  0