Stats preview card component

Community feedback

• 

  P

  Andrey•4,320

  @dar-ju

  Posted 22 days ago

  Hi, Tetiana!

  Awesome work!

  What I would change:

  • the picture is not just some abstract image or figures, it is a fully-content image. Content for a search engine can only be transmitted via the <img> tag with a description in alt. In more complex works, you will always need to ask yourself - is this a content image or just a decorative element? And then decide how to position it correctly
  • at a screen width of 760px, the block seems too narrow, you can remove the fixed width and make side paddings
  • the card__stats block is more of a list, use ul
  • there is an important point about the height of the blocks, use a fixed height very carefully. Imagine that there will be 5 times more text in the card - add it and check what happens. The text will simply go beyond the block. And your layout should be resistant to changes

  Otherwise, everything is great, keep up the good work!

  10

  Marked as helpful
• 

  Yacoub AlDweik•3,110

  @YacoubDweik

  Posted 22 days ago

  Hey! good job!

  .card__content > .content__title and not .card__title

  12