NFT preview card component

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  👾Hello Dave, Congratulations on completing this challenge!

  Amazing solution! I’ve just opened the solution’s live site and I liked the job you’ve done a lot. I’ve some suggestions for you:

  1.Use units as rem or em instead of px to improve your performance by resizing fonts between different screens and devices. These units are better to make your website more accessible. REM does not just apply to font size, but to all sizes as well.

  2.The box-shadow is a bit too strong, this is due the opacity and blur. The secret to create a perfect and smooth shadow is to have low values for opacity and increase blur try this value instead: box-shadow: 12px 7px 20px 6px rgb(57 75 84 / 8%);

  3.If you’re not familiar to box-shadow you can use this site to create the shadow design and then just drop the code into the CSS: https://html-css-js.com/css/generator/box-shadow/

  ✌️ I hope this helps you and happy coding!

  Marked as helpful

  0

  ---

  

  Dave•90

  @GalerinPosted about 2 years ago

  @correlucas I really appreciate your suggestions, I will keep them in mind for the next challenges. Thank you very much!

  2
• 

  Light Yagami•90

  @rakibhossainrajuPosted about 2 years ago

  Really good work man!! Except the box-shadow everything was almost the same. And i have to mention how nicely you have structure your code. In-fact I refactor my code after watching how well you code management is. Hopefully you will do great. GOOD LUCK !!

  0

  ---

  

  Dave•90

  @GalerinPosted about 2 years ago

  @Zubair1020 Thank you very much. I am glad to hear that you found my structure useful. Good luck to you too!

  1