NFT Product pagee

Community feedback

• 

  P

  visualdennis•8,375

  @visualdennissPosted over 1 year ago

  Hey there,

  Nice work. By default images are display: inline, like text, therefore you need to have it display: block; to get rid of the little space below that (if you look carefully, overlay and image don't match at the very bottom)

  I recommend using a CSS Reset like this at the beginning of each project, so you dont face such issues: Here is great one: https://www.joshwcomeau.com/css/custom-css-reset/

  BONUS TIP: Add a transition like transition: all .4s ease; to .card__image-cover for a smooth change in opacity.

  Hope you find this feedback helpful!

  1
• 

  Ecem Gokdogan•9,380

  @ecemgoPosted over 1 year ago

  Some recommendations regarding your code that could be of interest to you.

  Your accessibility report states that images must have alternate text. So, in order to fix the accessibility issues, your <img> tag should include "alt" = attribute.

  Hope I am helpful. :)

  0