Stats preview card component

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hey, Sukhanpreet! 👋

  Congratulations on completing this challenge! 🎉

  I have some feedback on this solution.

  • For the stat, I recommend making it as a list element instead. Also, the number statistic should not be h2. Try writing the HTML without CSS. This will help you understand what is the best element for this. In this case, a list would be a good choice.
    • 10k+ companies
    • 314 templates
    • 12m+ queries
  • For the purple overlay on the image, I would recommend using mix-blend-mode instead of filter. Not only, it is much easier to implement but also it is possible to make it looks closer to the design. Then, try to adjust the opacity of the img.

  One last thing that I would like to suggest is to remove the commented code. If another developer (it can be you in the future) wants to improve this solution, he/she might get confused about whether or not the commented code should be used or deleted.

  That's it! I hope this helps. 😊

  Marked as helpful

  1

  ---

  

  SK•130

  @SukhanpreetPosted over 2 years ago

  @vanzasetia Thanks for the feedback. :)

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @Sukhanpreet No problem! 😄

  0