Nft Preview Card Solution

Community feedback

• 

  gfunk77•1,200

  @gfunk77Posted 12 months ago

  This is great. Congratulations on your solution. I would suggest a few very minor points:

  • You have body: {margin: 0} , you can add margin and padding to your reset:

  *, ::after, ::before {
  margin: 0;
  padding: 0;
  box-sizing: border-box
  }
  

  You card has a fixed width and height. While it's not an issue at all in this case, consider not using fixed values because they will immediately remove responsiveness. Again, not a big deal here but something to think about.

  I don't see the hover states. You can create an overlay on the image for that.

  Great job. I loved your solution!

  Marked as helpful

  1

  ---

  

  aabc24•220

  @aabc24Posted 12 months ago

  I didn't look at the active states. Thank you, @gfunk77

  0