NFT preview card solution

Community feedback

• 

  Purnima Kumar•390

  @purnimakumarrPosted over 2 years ago

  Hi Joy,

  Congrats on completing the challenge 🙌

  Regarding you question about hover effect: I reviewed your code and the way you added style for a hover state looks good and acceptable. I would suggest to always use transition property while working with hover states. It makes the hover effect more smooth and pleasent looking. In your code you can add transition: all 0.3s to the .image component and h3 tag.

  Some suggestion by me: I see there is 1 accessibility issue in your code i.e. beacuse you didn't use an h1 tag in your solution. The first heading of any website should always be wrapped by an h1 tag as it makes the the website more accessible. h1 as the name of the tag suggests is used for the first heading a website. So, you can replace your h3 tag with an h1

  Marked as helpful

  1