stats-preview-card-component-main

Community feedback

• 

  Faris P•2,810

  @FarisPalayiPosted over 3 years ago

  Looks nice and responsive 👍

  A few suggestions are:

  • Set the image's alt tag to alt="" since the image here doesn't provide any extra context or info to users. And so that assistive devices can ignore them.
  • Try using a smaller media query breakpoint because, even though currently I'm using my laptop to see your solution, I'm getting the mobile version of the site.
  • Also, on mobile, the attribution text is on top of the card, so try to change it if you can😊

  That's all from me. Have fun coding ✨

  Marked as helpful

  1

  ---

  

  leonardo reis•290

  @leonardo9245Posted over 3 years ago

  @FarisPalayi Hi, thanks for your feedback, I made the changes, could you check again?

  thankful

  1

  

  Faris P•2,810

  @FarisPalayiPosted over 3 years ago

  @leonardo9245 Looks great👍 and layout is responding really well. Well done 👏

  btw, the font-family of the text under stats(for eg: "companies") is lexend deca not Inter😀

  One more suggestion would be to change those h2 tags to <strong> or <span> because those are technically not heading for anything. Take a look at Semantic HTML

  As always, have fun coding 🌟

  1