NFT preview card component

Community feedback

• 

  Aakash Verma•9,520

  @skyv26

  Posted over 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

  11

  Marked as helpful