Frontend Mentor | NFT preview card component (html and scss)

Community feedback

• 

  Macdeesh•610

  @macdeeshPosted over 2 years ago

  Hello Mohamed,

  That looks pretty good except for a few things:

  1- What do you expect when you see a hover effect? You know something is clickable, right? That means it's an interactive element. Every place you see a hover style, you need to include an interactive element, like an anchor tag or button. That is essential to fix.

  2- I would do the hover effect on this using pseudo-elements rather than adding extra in the HTML, but that's not essential. Also, for the other SVG icons you can use pseudo-elements like ::before or ::after, as the icons are all decorative.

  3- Don't forget to write a good alt text to your image.

  0