Solution for nft-preview-card-component-main challenge using html/css

Community feedback

• 

  Aakash Verma•6,710

  @skyv26Posted almost 3 years ago

  Hi! Yousef, i noticed some issues.

  1. When I clicked on image then cyan overlay not appeared with eye icon properly, you Overlay is behind the image so use zindex with position property in css will solve this issue

  2. In some mobile response your image part is going outside the scope (going upwards ) to topside.

  I hope you understand

  Keep coding

  Marked as helpful

  0

  ---

  

  Yousef Khalid•580

  @YousefKhalid-iqPosted almost 3 years ago

  @skyv26 Thanks for the feedback, I added a z-index to the image preview that is higher than the z-index for the image itself, and I didn't see what you meant with the 2. point, I tried galaxy fold's dimension and 375px and it looked centered for me.

  2

  

  Aakash Verma•6,710

  @skyv26Posted almost 3 years ago

  @YousefKhalid-iq 😂😂 My friend, I can understand that you completed your work according to requirement. But in web development, every design made by you, never consider what is right or wrong by yourself.

  What you have done is come under the bad user experience. I made so many mistakes, I still making mistakes (unintentionally) but I learnt from mistakes and I love learning.

  Good Luck 🤞

  1

  

  Yousef Khalid•580

  @YousefKhalid-iqPosted almost 3 years ago

  @skyv26 Thanks a lot, yeah you're right 😁

  2