NFT preview card component

Community feedback

• 

  P

  Remus D. Buhaianu•3,125

  @MiculinoPosted almost 3 years ago

  Hey @TejaswiniLabade, good job on completing this challenge!

  I'm impressed with how similar to the original design your final solution looks - well done!

  As @denielden already suggested, you can consider adding the transition property for your hover effects, mainly the image hover effect as it can give it that extra nice detail ;)

  Had a look at your Github repo and you should definitely consider looking into BEM methodology. Also, some parts of your code require a bit of formatting so they can be more readable.

  Overall, well done. Keep up the good work!

  Marked as helpful

  1
• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

  Hi Tejaswini, great work!

  You can remove the margins from the .card class because they are superfluous ... flexbox already has it centered.

  Try to use transition property to all element with :hover state ;)

  Happy coding!

  Marked as helpful

  0