NFT preview card component

Community feedback

• 

  Aakash Verma•6,710

  @skyv26Posted almost 3 years ago

  Hi! Mortada, I was just looking at challenge submission and then I saw your submission and willingly I check your work. You did it really good, I mean I liked your hover effect and the way the cyan overlay with eye appeared with animation is really too good but here are some suggestions for you.

  1. Just make image hover state visibile when user only hover over image not on card.

  2. Add cursor: pointer; property on all hover state.

  3. In mobile view your card is going outside the viewport. as user scroll down, it can be seen easily. Reason behind is you have used 100vh somewhere in your elements property and It also need to fix soon.

  I hope you liked my suggestions and soon will fix issues.

  Good Luck

  Marked as helpful

  1

  ---

  

  Mortada Emad•230

  @rida192Posted almost 3 years ago

  @skyv26 thanks alot, your feedback are very helpful, i will try to do my best!

  1