NFT Preview Card Solution

Community feedback

• 

  Prabhash Ranjan•2,540

  @besttlookkPosted over 2 years ago

  HI, Great work. Few thinks you can change to make this even better.

  • Center the content w.r.t the screen.Also when there is content only for 1 view-port size disable the scrolling. Here is how you can do it:

  body{
     display:flex;
     justif-content:center;
     align-items:center;
     min-height:100vh;
     overflow:hidden
  

  • Hover effect is too abrupt. To add smooth transition. Use transition-property.

  Apart from these things rest looks okay to me.

  Keep up the good work

  Good Luck

  Happy Coding

  Marked as helpful

  0

  ---

  

  Kieera Basey•40

  @kbaseyPosted over 2 years ago

  @besttlookk Thank you for the feedback. I applied your suggestions and I agree that disabling the scrolling makes a big difference and I especially enjoy the smooth transition using transition-property. Thank you again!

  1

  

  Prabhash Ranjan•2,540

  @besttlookkPosted over 2 years ago

  @kbasey Happy to help :)

  Btw while making the changed you messed something up. Card on hover is not perfectl aligned. Plus reduce the duration of transition. its too slow , 500ms would do the job. Also add "cursor:pointer" on hover effect.

  Good Luck,

  Happy coding

  0