Responsive NFT preview card component

Community feedback

• 

  P

  Remus D. Buhaianu•3,125

  @MiculinoPosted almost 3 years ago

  Congrats on completing the challenge, @SyllightTheway !

  Your solution looks quite good overall. Here are a few suggestions based on your final solution that I hope will be useful to you:

  • The .line element shouldn't stretch fully from one end to another - consider adding some horizontal padding to it

  • The .name element is too much to the right and it's not vertically centered with the avatar image

  • The Etherum img has edges that are a bit too round. Also add some more padding around it

  • Your image overlay effect on hover doesn't work / exist. Here's one way to achieve it https://www.w3schools.com/howto/howto_css_image_overlay.asp You can also check my solution for this challenge and see how I've achieved the effect using ::after and ::before pseudo-classes

  Hope this helps. Keep up the good work!

  Marked as helpful

  2

  ---

  

  Syllight•35

  @SyllightThewayPosted almost 3 years ago

  @Remus432 Thank you so much for suggestions!

  1